search

aiidaplugins / aiida-lammps

LAMMPS plugin for AiiDA
https://aiida-lammps.readthedocs.io
MIT License
22 stars 14 forks source link

Release v1.0.0 #92

Closed JPchico closed 9 months ago

JPchico commented 9 months ago

Bump version number to 1.0.0

Updating the Changelog.

Removing the documentation under development banner.

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (47b1a4d) 85.51% compared to head (df152b8) 85.51%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #92 +/- ## ======================================== Coverage 85.51% 85.51% ======================================== Files 19 19 Lines 1622 1622 ======================================== Hits 1387 1387 Misses 235 235 ``` | [Flag](https://app.codecov.io/gh/aiidaplugins/aiida-lammps/pull/92/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aiidaplugins) | Coverage Ξ” | | |---|---|---| | [pytests](https://app.codecov.io/gh/aiidaplugins/aiida-lammps/pull/92/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aiidaplugins) | `85.51% <100.00%> (ΓΈ)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aiidaplugins#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

JPchico commented 9 months ago

@chrisjsewell , @sphuber any comments before merging this? I assume that I can make the tag and release directly from the release github tab right?

chrisjsewell commented 9 months ago

All looks good cheers πŸŽ‰ You need to merge develop in to master before releasing. You just make the release from GitHub and it creates the tag for you.

Then we can delete develop/development branches and work straight from master.

chrisjsewell commented 9 months ago

After you merge the PR, do the following in your local clone ...

Nope πŸ‘Ž πŸ˜‰ just do it from GitHub

JPchico commented 9 months ago

Thank you for your help @chrisjsewell and @sphuber ! The release is done! If something comes up just let me know!

chrisjsewell commented 9 months ago

Think the deployment workflow needs updating, I can do it in a minute