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

Add auto-publish script #171

Merged
merged 9 commits into from
Dec 3, 2024
Merged

Add auto-publish script #171

merged 9 commits into from
Dec 3, 2024

Conversation

plehegar
Copy link
Contributor

@plehegar plehegar commented Sep 6, 2024

Ideally, we should point to a link where TAG and PING are agreeing to do this. Having said that, this pull request could serve as such.

@torgo @plinss @npdoty @pes10k

Copy link
Contributor

@jyasskin jyasskin left a comment

Choose a reason for hiding this comment

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

A couple changes in order to avoid having two deploy scripts:

.github/workflows/auto-publish.yml Outdated Show resolved Hide resolved
.github/workflows/auto-publish.yml Show resolved Hide resolved
.github/workflows/auto-publish.yml Show resolved Hide resolved
@plehegar
Copy link
Contributor Author

Can this be merged?

@pes10k
Copy link
Contributor

pes10k commented Nov 23, 2024

This looks good to me. If it looks good to @hober too then i'm good with merging

@npdoty
Copy link

npdoty commented Dec 2, 2024

@pes10k pes10k merged commit c5ffa65 into w3ctag:main Dec 3, 2024
1 check passed
github-actions bot added a commit that referenced this pull request Dec 3, 2024
SHA: c5ffa65
Reason: push, by pes10k

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@pes10k
Copy link
Contributor

pes10k commented Dec 3, 2024

@ylafon , wanted to touch bases with you as follow up here at @hober 's suggestion. Do you have any thoughts or concerns, or suggestions for further improvement

github-actions bot added a commit to MaxMood96/security-questionnaire that referenced this pull request Dec 3, 2024
SHA: c5ffa65
Reason: push, by pull[bot]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@ylafon
Copy link
Member

ylafon commented Dec 4, 2024

The SoTD produced by that script has a different licensing requirement section, I don't think there is an issue now, but it was some times ago. If no-one complains about it, I can only be happy that auto-publishing is working now!

github-actions bot added a commit to plehegar/security-questionnaire that referenced this pull request Dec 5, 2024
SHA: c5ffa65
Reason: push, by plehegar

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

use automatic publishing
5 participants