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

Fix release notes + version in docs #742

Merged
merged 6 commits into from
Apr 3, 2023
Merged

Fix release notes + version in docs #742

merged 6 commits into from
Apr 3, 2023

Conversation

jmduarte
Copy link
Member

@jmduarte jmduarte commented Apr 1, 2023

Fix the autogeneration of release notes.

Also fix version number.

Type of change

  • Documentation update

Checklist

  • I have read the guidelines for contributing.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have installed and run pre-commit on the files I edited or added.

@jmduarte
Copy link
Member Author

jmduarte commented Apr 1, 2023

OK, this fixes the autogenerated release notes, which we can check by downloading the build as an artifact: https://github.com/fastmachinelearning/hls4ml/suites/11957657003/artifacts/627077421

the versioning is still off, but that's a different problem...

@jmduarte jmduarte requested a review from jmitrevs April 1, 2023 01:45
@jmduarte jmduarte added the bug label Apr 1, 2023
@jmduarte jmduarte added this to the v0.7.0 milestone Apr 1, 2023
@jmduarte jmduarte changed the title Fix release notes Fix release notes + version in docs Apr 1, 2023
@jmduarte
Copy link
Member Author

jmduarte commented Apr 1, 2023

OK it's all working now (including version number)! check the latest artifact!

https://github.com/fastmachinelearning/hls4ml/suites/11957927384/artifacts/627099082

resolution found via: actions/checkout#1049

@vloncar vloncar merged commit 4ade873 into main Apr 3, 2023
2 checks passed
@vloncar vloncar deleted the fix_release_notes branch April 3, 2023 16:06
JanFSchulte pushed a commit to JanFSchulte/hls4ml that referenced this pull request May 23, 2023
* Fix release notes?

* upload artifact

* try global git config

* try more tricks

* don't take ownership

* Edit on GitHub
calad0i pushed a commit to calad0i/hls4ml that referenced this pull request Jul 1, 2023
* Fix release notes?

* upload artifact

* try global git config

* try more tricks

* don't take ownership

* Edit on GitHub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants