Skip to content

94 doc release procedure#103

Open
rogerkuou wants to merge 5 commits into
mainfrom
94_doc_release_procedure
Open

94 doc release procedure#103
rogerkuou wants to merge 5 commits into
mainfrom
94_doc_release_procedure

Conversation

@rogerkuou

Copy link
Copy Markdown
Member

fix #94

  • Update Changelog
  • Add a GitHub issue template for release checklist
  • Add an agent skill for generating changelog infor
  • Update JOSS paper AI usage disclosure section.

@sonarqubecloud

Copy link
Copy Markdown

@rogerkuou

Copy link
Copy Markdown
Member Author

Hi @SarahAlidoost , when you have time, can you help me review this PR?

This PR is to fix a comment we got from the JOSS review, where the reviewer asked us to improve the Change Log, create templates to standardize the release process.

While working on these suggestions, I think I can go one step further and created an agent skill to generate draft CHANGELOG info. This is the part I specifically want to ask for your input.

You can run in GitHub Copilot /release-changelog v1.2.4 to locally test the agent skill for the next release. This is also documented in the GitHub issue template.

@SarahAlidoost SarahAlidoost left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rogerkuou looks good! 👍 Nice to see how SKILLS can be created for coding agents and how tasks can be done by agents. I haven't run the agent though, but I could run the script. I left some comments.


---

I will make a new release by preforming the following procedures:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
I will make a new release by preforming the following procedures:
I will make a new release by preforming the following steps:

Comment on lines +19 to +20
- The agent skill can be executed in GenAI prompt, e.g. GitHub Copilot Chat, by `/release-changelog vX.Y.Z` command, where `vX.Y.Z` is the new version label.
- Alternatively, one can run the helper script `bash .github/skills/release-changelog/scripts/generate_release_section.sh vX.Y.Z` to get bullets for the new release section. No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are documentation on how the agent skill can be called. I suggest moving these two lines to the CONTRIBUTING.md under the section "Making a release".

- [ ] Step3: Create release/tag at https://github.com/TUDelftGeodesy/sarxarray/releases

Hints:
- Step 1 and 2 can be done semi-automatically by calling agent skill in [release-changelog](.github/skills/release-changelog). However the changes will always be reviewed and commited by human.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a documentation on how to call the agent if so, please add a link here.

Comment on lines +13 to +16
- [ ] Step1: Update version in pyproject.toml
- [ ] Step2: Update `docs/CHANGELOG.md` with new change information
- [ ] Step3: Create release/tag at https://github.com/TUDelftGeodesy/sarxarray/releases

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In addition to this issue template, release guide should be added to the CONTRIBUTING.md under a section "Making a release", see example.


## When To Use

- You just created a new release tag and want changelog content.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the changelog be created/update before the release tag?

- bash .github/skills/release-changelog/scripts/generate_release_section.sh vX.Y.Z vX.Y.(Z-1)

Notes:
- This supports changelog-first workflows where docs are built from release-triggered actions.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence is not clear.

Comment thread docs/CHANGELOG.md

See [releases](https://github.com/TUDelftGeodesy/sarxarray/releases) in the sarxarray repository for more details.

The information in this file can be generated automatically by LLM using the [release-changelog skill](.github/skills/release-changelog/SKILL.md). However, the generated content has always been reviewed and if necessary, edited by a human.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The information in this file can be generated automatically by LLM using the [release-changelog skill](.github/skills/release-changelog/SKILL.md). However, the generated content has always been reviewed and if necessary, edited by a human.
The information in this file can be generated automatically by LLM using the [release-changelog skill](../.github/skills/release-changelog/SKILL.md). However, the generated content has always been reviewed and if necessary, edited by a human.

- [ ] Step3: Create release/tag at https://github.com/TUDelftGeodesy/sarxarray/releases

Hints:
- Step 1 and 2 can be done semi-automatically by calling agent skill in [release-changelog](.github/skills/release-changelog). However the changes will always be reviewed and commited by human.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Step 1 and 2 can be done semi-automatically by calling agent skill in [release-changelog](.github/skills/release-changelog). However the changes will always be reviewed and commited by human.
- Step 1 and 2 can be done semi-automatically by calling agent skill, see [release-changelog](.github/skills/release-changelog). However the changes will always be reviewed and commited by human.

Comment thread paper/paper.md

In writing the software documentation and the JOSS paper, GPT-5 was used for language improvements for grammar correction and style edits.

Specifically, in the documentation process, an agent skill (under path `.github/skills/release-changelog`) is used to generate changelog entries from git tags and commit history. The generated content was always reviewed and edited by a human before being committed to the repository.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Specifically, in the documentation process, an agent skill (under path `.github/skills/release-changelog`) is used to generate changelog entries from git tags and commit history. The generated content was always reviewed and edited by a human before being committed to the repository.
Specifically, in the documentation process, an agent skill (under path `../.github/skills/release-changelog`) is used to generate changelog entries from git tags and commit history. The generated content was always reviewed and edited by a human before being committed to the repository.

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.

Document release process and include actual release notes in Changelog file

2 participants