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

Normalize SE-0451 author and previous proposal fields #2622

Merged

Conversation

dempseyatgithub
Copy link
Contributor

@dempseyatgithub dempseyatgithub commented Nov 24, 2024

The author field of SE-0451 includes a reference to the previous proposal that SE-0451 was based on.

The Author field should only include people. The reference to a previous proposal should be in the Previous Proposal field.

Note also that the author of the previous proposal (SE-0275) is properly credited in that proposal and does not need to be included as part of the current proposal (SE-0451) unless the author of SE-0451 wishes to include them as a co-author.

This PR normalizes the Author and Previous Proposal fields and closes swiftlang/swift-org-website#856.

@rjmccall
Copy link
Contributor

rjmccall commented Dec 2, 2024

@allevato, would you like to include Alfredo as a co-author? You should probably only do so if he's interested in still participating.

@allevato
Copy link
Member

allevato commented Dec 3, 2024

I reached out to Alfredo and he said it didn't matter to him either way. I just wanted to make sure he was properly attributed, and this proposal is effectively a full rewrite from the ground up, so I think this PR looks good. Thanks @dempseyatgithub !

@allevato allevato merged commit 23823bb into swiftlang:main Dec 3, 2024
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.

SE-0451 has an additional author by mistake
3 participants