Skip to content
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

Change terms as suggested in #2015 #2075

Merged
merged 3 commits into from May 29, 2018
Merged

Conversation

chsanch
Copy link
Contributor

@chsanch chsanch commented May 29, 2018

Terms fixed: runtime, shorthand, lookahead, lookbehind

The problem

Looking for handles to solve #761, I have found that file handle appear at least in that form or in the filehandle form. We should try and normalize terminology across all the documents, and this is only an example. When normalization is done, we should turn it into a test later on.

Solution provided

Changes applied:
runtime, run-time → runtime
short-hand, shorthand →shorthand
look-ahead, lookahead →lookahead
look-behind, lookbehind →lookbehind

Terms fixed: runtime, shorthand, lookahead, lookbehind
Copy link
Contributor

@JJ JJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is amazing. Just that tiny change and we're good to go.

@@ -1779,7 +1779,7 @@ also modify the other.
X<|smartmatch operator>
=head2 infix C«~~»

The X<smart-match operator> aliases the left-hand side to C<$_>, then evaluates
The X<smartmatch operator> aliases the left-hand side to C<$_>, then evaluates
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to leave this one as it is, since it's used for indexing. If you don't, just eliminate the X<> since it's already indexed a few lines above.

chsanch and others added 2 commits May 29, 2018 14:39
Revert change added in the pull request: smartmatch to smart-match
@rafaelschipiura rafaelschipiura merged commit de6693e into Raku:master May 29, 2018
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.

None yet

3 participants