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

Script to generate the excerpt of the GH release #566

Merged
merged 3 commits into from
Aug 22, 2023

Conversation

sholderbach
Copy link
Member

This will add a list of PR authors so they will be shown with a profile
image at the end of the GH release.

This will add a list of PR authors so they will be shown with a profile
image at the end of the GH release.
Copy link
Member

@amtoine amtoine left a comment

Choose a reason for hiding this comment

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

you say "they will be shown with a profile image" but i'm not sure i see that in the output of the script 😕
am i missing something? 😇

oooh wait, you meant the real GitHub release, like this one!!
yeah we see faces at the bottom, this is nice 👌

Comment on lines 10 to 13
let date = (
if $date == null { (date now) - 3wk + 1day } else { $date }
| date format "%Y-%m-%d"
)
Copy link
Member

Choose a reason for hiding this comment

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

as in #561, i would

  • the default command
  • switch to the 4-week schedule

😋

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah right I copied the code on main without looking to deep into that, and yesterday I was running it with explicit dates anyway.

Copy link
Member

Choose a reason for hiding this comment

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

no worries 😉

the question is just: do we fix that here or in another PR? 😋

make_release/release-note/gh-release-excerpt Outdated Show resolved Hide resolved
sholderbach and others added 2 commits July 31, 2023 21:40
Co-authored-by: Antoine Stevan <44101798+amtoine@users.noreply.github.com>
@sholderbach sholderbach merged commit f2e35e1 into nushell:main Aug 22, 2023
@sholderbach sholderbach deleted the script-gh-release-text branch August 22, 2023 22:42
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.

None yet

2 participants