The Wayback Machine - https://web.archive.org/web/20220301142815/https://github.com/pytorch/vision/pull/5457
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

Make small improvements to the release notes scripts #5457

Merged
merged 6 commits into from Feb 23, 2022

Conversation

jdsgomes
Copy link
Contributor

@jdsgomes jdsgomes commented Feb 22, 2022

Add release tags as parameters, exclude prototype stuff from release notes and
automate contributors list

Add release tags as parameters, exclude prototype stuff from release notes and
automate contributors list
@facebook-github-bot
Copy link

@facebook-github-bot facebook-github-bot commented Feb 22, 2022

💊 CI failures summary and remediations

As of commit 96e2785 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.
Copy link
Member

@NicolasHug NicolasHug left a comment

Thanks @jdsgomes

# In[12]:

# Generate list of authors
command_to_run = f"{{ git shortlog -s {previous_release}..{current_release} | cut -f2- & git log -s {previous_release}..{current_release} | grep Co-authored | cut -f2- -d: | cut -f1 -d\< | sed 's/^ *//;s/ *$//' ; }} | sort --ignore-case | uniq | tr '\n' ';' | sed 's/;/, /g;s/, $//' | fold -s"
Copy link
Member

@datumbox datumbox Feb 23, 2022

Choose a reason for hiding this comment

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

The linter is complaining about this line. It might be due to a change made yesterday to run temporarily on root. This was patched today so let's merge main and retry.

@jdsgomes jdsgomes merged commit 8097370 into pytorch:main Feb 23, 2022
5 of 94 checks passed
@github-actions
Copy link

@github-actions github-actions bot commented Feb 23, 2022

Hey @jdsgomes!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

facebook-github-bot added a commit that referenced this issue Feb 25, 2022
Summary:
* Make small improvements to the release notes scripts

Add release tags as parameters, exclude prototype stuff from release notes and
automate contributors list

Reviewed By: jdsgomes

Differential Revision: D34475323

fbshipit-source-id: 4a852e86a4ddff2dd60dc64d5dbe0cd92127aecb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment