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

Assorted fixes #106

Merged
merged 6 commits into from
Jul 5, 2021
Merged

Assorted fixes #106

merged 6 commits into from
Jul 5, 2021

Conversation

waldyrious
Copy link
Collaborator

Follow-ups to PR #100:

  • 2f7991d fixes a copy/paste mistake in commit 5044fd7
  • 8c8edbf updates the documentation to match the PR changes
  • b935d45 expands documentation for the token() function, as discussed here

Other changes:

All of these fixes were authored by @Xymph, and are being extracted from #102 to keep that one focused on the release preparations.

@waldyrious waldyrious requested a review from Xymph July 4, 2021 21:13
@waldyrious waldyrious added the Bug label Jul 4, 2021
CHANGELOG.md Outdated Show resolved Hide resolved
@Xymph
Copy link
Collaborator

Xymph commented Jul 4, 2021

Other than the one minor remark, everything looks correct and complete.

@waldyrious
Copy link
Collaborator Author

waldyrious commented Jul 4, 2021

Updated per your suggestion. Feel free to merge whenever is convenient to you.

By the way, I am out of time for today, but I should be able to rebase #102 tomorrow.

@Xymph Xymph merged commit 1ed84e5 into master Jul 5, 2021
@Xymph
Copy link
Collaborator

Xymph commented Jul 5, 2021

That's a wrap. Great progress yesterday.

@waldyrious waldyrious deleted the assorted-fixes branch July 5, 2021 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants