Skip to content

Conversation

@imatsoup
Copy link
Collaborator

About the PR

Made changes we'd been talking about to publish workflows and added changelog.yml (ripped from deltav) workflow file to make changelogs easier.

Why / Balance

I wish to put an end to @arenaconspiracy's suffering

Technical details

Media

Requirements

Breaking changes

Changelog

🆑

  • add: Automated changelog workflows to make changelogs easier to do.

Co-Authored-By: Debug <49997488+debugok@users.noreply.github.com>
@github-actions github-actions bot added S: Untriaged Status: Needs appropriate labels. size/S S: Needs Review Status: This PR requires new reviews before it can be merged. Changes: No C# changes: Might require no knowledge of C#. and removed S: Needs Review Status: This PR requires new reviews before it can be merged. S: Untriaged Status: Needs appropriate labels. size/S labels Sep 26, 2025
@imatsoup
Copy link
Collaborator Author

Note: Schema validators failing is unrelated. Should maybe consider turning them off too while those are broken upstream.

@arenaconspiracy arenaconspiracy self-requested a review September 27, 2025 11:22
@github-actions github-actions bot added the S: Needs Review Status: This PR requires new reviews before it can be merged. label Sep 27, 2025
- name: Generate Changelog
run: |
cd "Tools/changelog"
node changelog.js

Choose a reason for hiding this comment

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

This is calling a script that we don't actually have, as far as I can tell. See DeltaV-Station/Delta-v#11 having added Tools/Changelogs/changelog.js

types: [closed]

env:
GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}

Choose a reason for hiding this comment

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

Do we have these secrets set up? I'm not 100% on where they're supposed to be stored.

Copy link

@arenaconspiracy arenaconspiracy left a comment

Choose a reason for hiding this comment

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

Accidentally pressed "single comment" instead of starting a review on prior comments, whoops. They're there though.

@arenaconspiracy arenaconspiracy added enhancement New feature or request P1: High Priority: Higher priority than other items, but isn’t an emergency. S: Awaiting Changes Status: Changes are required before another review can happen. labels Sep 27, 2025
@imatsoup imatsoup marked this pull request as draft September 27, 2025 16:52
@imatsoup
Copy link
Collaborator Author

Converted to a draft until secrets and vars are worked out

@github-actions github-actions bot added size/M and removed size/S labels Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: No C# changes: Might require no knowledge of C#. enhancement New feature or request P1: High Priority: Higher priority than other items, but isn’t an emergency. S: Awaiting Changes Status: Changes are required before another review can happen. S: Needs Review Status: This PR requires new reviews before it can be merged. size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants