Skip to content

Add the DICTIONARY_AUTHOR loop#317

Merged
rowlesmr merged 5 commits into
COMCIFS:masterfrom
rowlesmr:add-dictionary-author-list
Jul 7, 2026
Merged

Add the DICTIONARY_AUTHOR loop#317
rowlesmr merged 5 commits into
COMCIFS:masterfrom
rowlesmr:add-dictionary-author-list

Conversation

@rowlesmr

@rowlesmr rowlesmr commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

From #266

In no way can this be considered complete or in the correct order. I just put all the names in the order I initially wrote them down in.

Emails intentionally not included until explicit permission is given. I have also not included ORCIDs unless previously used in core.

rowlesmr added 2 commits July 7, 2026 20:33
in no way is this complete or in the correct order. Emails intentionally not included until explicit permission is given. I have also not included ORCIDs unless previously used in core.
@vaitkus

vaitkus commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Thank you for the PR. Given the previous discussion in the related issue, I think that this list can be merged as is and further adjusted during the editorial process.

We can leave the related issue open, however, merging this will resolve the semantic issue warning that currently appears on all PRs.

@rowlesmr rowlesmr marked this pull request as ready for review July 7, 2026 12:50
@rowlesmr

rowlesmr commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator Author

No dramas. Made non-draft for review, and merging this will no longer close the issue.

@rowlesmr rowlesmr merged commit 1fc2135 into COMCIFS:master Jul 7, 2026
3 checks passed
@rowlesmr rowlesmr deleted the add-dictionary-author-list branch July 7, 2026 14:08
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.

2 participants