cargo-mutants icon indicating copy to clipboard operation
cargo-mutants copied to clipboard

Published 20 hours ago • verified publisher icon sourcefrog

Setup CITATION.cff

Open kevinmatthes opened this issue 1 year ago • 2 comments

This PR adds a basic setup for citation metadata powered by CITATION.cff, as discussed in #210. (Please note that any papers citing this project using these information are yet to be written. 😉)

kevinmatthes avatar Dec 29 '23 23:12 kevinmatthes

The next step is to discuss the actual implementation of the update automation for the CITATION.cff. The update steps are rather straightforward: replacing the old version with the new one and updating the release date to "today".

Is there any workflow I can put the two required sed commands or do you use cargo-release?

kevinmatthes avatar Dec 29 '23 23:12 kevinmatthes

The next step is to discuss the actual implementation of the update automation for the CITATION.cff. The update steps are rather straightforward: replacing the old version with the new one and updating the release date to "today".

Is there any workflow I can put the two required sed commands or do you use cargo-release?

I have just been doing it manually and thinking that I should find a tool to automate it. Rather than adding sed commands let's do that now...? Maybe cargo-release is a good choice. It looks like we could configure it to update this file. Filed #213

sourcefrog avatar Jan 01 '24 15:01 sourcefrog

@sourcefrog

You can go ahead and merge this PR. I will submit a separate PR for the automatic maintenance (#213).

kevinmatthes avatar Jan 06 '24 14:01 kevinmatthes

The tests are not running because the Docker image for the validation tool is not a trusted Action, yet. How to fix this problem, @sourcefrog?

kevinmatthes avatar Jan 06 '24 19:01 kevinmatthes

Thanks for diagnosing it. I can add it to the allowlist.

sourcefrog avatar Jan 06 '24 19:01 sourcefrog

I will close and reopen this PR to have the tests run once again.

kevinmatthes avatar Jan 07 '24 13:01 kevinmatthes

I looked at the Actions docs and I couldn't see how to allow Docker containers.

On the whole this doesn't seem like a file that is likely to be edited very often at all (other than version bumps) and so it's not really important to have it automatically validated. So, could you remove that for now and we'll just add the file?

sourcefrog avatar Jan 07 '24 16:01 sourcefrog

Yes, no problem. I will remove the test. :+1:

kevinmatthes avatar Jan 07 '24 19:01 kevinmatthes

I just updated the version and date-released fields to refer to today's release. You can go ahead and merge this PR, I removed the test as you requested.

kevinmatthes avatar Jan 13 '24 15:01 kevinmatthes