Skip to content

[CI] fixes#2564

Merged
scott-brust merged 3 commits into
developfrom
fix/ci-issues
Oct 24, 2022
Merged

[CI] fixes#2564
scott-brust merged 3 commits into
developfrom
fix/ci-issues

Conversation

@avtolstoy

Copy link
Copy Markdown
Member

Description

  1. Fixes a problem with Slack notifications (there is a limit of 10 section items)
  2. Fixes issue with Artifacts build step for branches whose name starts with letters after t e.g. w.

Tested in #2560

Steps to Test

N/A

Example App

N/A

References

N/A


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@scott-brust

Copy link
Copy Markdown
Member

FYI I manually started CI and will approve when the build passes

@avtolstoy

Copy link
Copy Markdown
Member Author

Thanks, Scott. This resolves the CI problem for #2563 as well.

@avtolstoy

Copy link
Copy Markdown
Member Author

@scott-brust FYI the backport should not include Tracker M/P2 stuff, so only release-publish.sh fix is required.

@scott-brust

Copy link
Copy Markdown
Member

@scott-brust FYI the backport should not include Tracker M/P2 stuff, so only release-publish.sh fix is required.

👍 will pull this into a 4.x backport PR soon as its merged

@scott-brust
scott-brust merged commit d6365ae into develop Oct 24, 2022
@scott-brust
scott-brust deleted the fix/ci-issues branch October 24, 2022 18:45
scott-brust added a commit that referenced this pull request Oct 24, 2022
@scott-brust scott-brust added this to the 5.2.0 milestone Dec 16, 2022
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.

2 participants