Skip to content

FEDX-2835: Fixed issues from release#264

Merged
btr-rmconsole-5[bot] merged 1 commit intomasterfrom
fix_release
Mar 20, 2025
Merged

FEDX-2835: Fixed issues from release#264
btr-rmconsole-5[bot] merged 1 commit intomasterfrom
fix_release

Conversation

@matthewnitschke-wk
Copy link
Contributor

@matthewnitschke-wk matthewnitschke-wk commented Mar 20, 2025

FEDX-2835

Issue Status

Motivation

Release CI is failing due to missing CHANGELOG entries. There is a check in gha-dart-oss which prevents this, but its in a more recent version

This PR updates w_module to use the latest gha-dart-oss to implement the check to catch the release errors before the release PR is merged

Merge Checklist

  • CI passes

jobs:
checks:
uses: Workiva/gha-dart-oss/.github/workflows/checks.yaml@v0.1.3
uses: Workiva/gha-dart-oss/.github/workflows/checks.yaml@v0.1.7
Copy link
Contributor Author

Choose a reason for hiding this comment

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

v0.1.3 does not have the CHANGELOG check to address the missing changes

build_web_compilers: '>=3.0.0 <5.0.0'
dart_dev: ^4.0.0
dart_style: ^2.1.1
dependency_validator: ^3.2.2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

dep validator is installed globally

@matthewnitschke-wk matthewnitschke-wk marked this pull request as ready for review March 20, 2025 14:54
@bender-wk bender-wk changed the title Fixed issues from release FEDX-2835: Fixed issues from release Mar 20, 2025
@matthewnitschke-wk matthewnitschke-wk requested review from a team March 20, 2025 15:16
@matthewnitschke-wk
Copy link
Contributor Author

QA +1

CI does indeed pass, only changes are to CI

@matthewnitschke-wk
Copy link
Contributor Author

🚀 @Workiva/release-management-p 🚢

Copy link
Contributor

@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

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

+1 from RM

@btr-rmconsole-5 btr-rmconsole-5 bot merged commit cae551a into master Mar 20, 2025
10 checks passed
@btr-rmconsole-5 btr-rmconsole-5 bot deleted the fix_release branch March 20, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants