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.

Sign up for GitHub

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

Jump to bottom

Removed buildscript/buildexpression list sorting workarounds. #2831

Merged
merged 1 commit into from
Oct 19, 2023
Merged

Removed buildscript/buildexpression list sorting workarounds. #2831

merged 1 commit into from
Oct 19, 2023

Conversation

mitchell-as
Copy link
Contributor

@mitchell-as mitchell-as commented Oct 18, 2023 •
edited by github-actions bot
Loading

StoryDX-1939 As a user, I can modify my Build Script in my text editor and the state tool will pickup the changes

Buildexpressions from the Platforms should have the same list order.

@mitchell-as
Removed buildscript/buildexpression list sorting workarounds.
3f79f2d 
Buildexpressions from the Platforms should have the same list order.
@mitchell-as mitchell-as reopened this Oct 18, 2023
@mitchell-as
Copy link
Contributor Author

The test failures are not due to this PR or PB. They are known, existing issues and the usual timeouts.

@mitchell-as mitchell-as marked this pull request as ready for review October 18, 2023 21:06
@mitchell-as mitchell-as reopened this Oct 19, 2023
@mitchell-as mitchell-as merged commit ed2803e into version/0-42-0-RC1 Oct 19, 2023
16 of 26 checks passed
@mitchell-as mitchell-as deleted the mitchell/dx-1939 branch October 19, 2023 17:11
as-builds added a commit that referenced this pull request Oct 19, 2023
@as-builds
Merge branch version/0-42-0-RC1 to adopt changes from PR #2831
e04270f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MDrakos MDrakos MDrakos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mitchell-as @MDrakos