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

Bibliography formatting fixes #7507

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

mhoemmen
Copy link
Contributor

Fix spelling and formatting in bibliography.

These fixes were separated from PR #7504 per reviewer request.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 19, 2024

All done with this?

Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

Please rephrase the commit descriptions: https://github.com/cplusplus/draft/wiki/Commit-message-format

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Dec 19, 2024
source/back.tex Outdated Show resolved Hide resolved
@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 19, 2024

Please rephrase the commit descriptions: https://github.com/cplusplus/draft/wiki/Commit-message-format

Don't worry, I'll do that when squashing here.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 19, 2024

Once it's in, you can rebase and squash the other PR.

Fix spelling of one author's name.  Add missing commas,
and extra spaces after a period ending authors' abbreviated
first or middle names.
@mhoemmen
Copy link
Contributor Author

@tkoeppe wrote:

All done with this?

All done; thanks for asking!

@jensmaurer wrote:

Please rephrase the commit descriptions....

Done!

@jensmaurer jensmaurer merged commit 561a4d8 into cplusplus:main Dec 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Changes to the wording or approach have been requested and not yet applied.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants