8000 Cleanup and additional fixes regarding procedures by soc · Pull Request #212 · scala/docs.scala-lang · GitHub
[go: up one dir, main page]

Skip to content

Cleanup and additional fixes regarding procedures #212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 9, 2013
Merged

Cleanup and additional fixes regarding procedures #212

merged 1 commit into from
Jul 9, 2013

Conversation

soc
Copy link
Contributor
@soc soc commented Jun 27, 2013

No description provided.

@soc
Copy link
Contributor Author
soc commented Jun 27, 2013

@adriaanm Could you have a look?

@soc
Copy link
Contributor Author
soc commented Jul 9, 2013

Ping?

@adriaanm
Copy link
Contributor
adriaanm commented Jul 9, 2013

Sorry I missed this -- usually EPFL takes care of this project.

@adriaanm
Copy link
Contributor
adriaanm commented Jul 9, 2013

LGTM

adriaanm added a commit that referenced this pull request Jul 9, 2013
Cleanup and additional fixes regarding procedures
@adriaanm adriaanm merged commit 9f23ac1 into scala:master Jul 9, 2013
@heathermiller
Copy link
Member

@adriaanm: Yep, usually I take care of it. Though you were the named reviewer, so I left this one for you.
Let me know if next time you want me to ping you personally, in case you don't monitor scala.github.com PRs.

@adriaanm
Copy link
Contributor

No problem -- it's all me. I was drowning in notifications.

@adriaanm
Copy link
Contributor

That said, an extra @-mention-ping is always welcome. They normally go straight to my inbox.

@heathermiller
Copy link
Member

Alrighty, will do next time! Thanks!

bishabosha pushed a commit to bishabosha/docs.scala-lang that referenced this pull request Mar 21, 2022
Update hacker guide: add more info, fix broken links.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0