Skip to content

SwiftSyntax: Commit gyb-generated files to master #27841

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

Merged
merged 3 commits into from
Oct 25, 2019

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Oct 22, 2019

This is a companion PR to swiftlang/swift-syntax#159. It adds the necessary flags to forward --swiftsyntax-verify-generated-files and adds a preset with which we can test SwiftSyntax in CI. At the moment SwiftSyntax is using a hard-baked command in CI.

…ed-files

This will allow us to set up a CI bot that can test that all generated
sources in SwiftSyntax are up-to-date.
…s all generated files

We no longer need to generate the files using gyb. Instead we should
check that the generated files are actually up-to-date.
@ahoppen
Copy link
Member Author

ahoppen commented Oct 23, 2019

@swift-ci Please test

@ahoppen ahoppen requested a review from nkcsgexi October 23, 2019 22:13
@ahoppen ahoppen changed the title WIP: SwiftSyntax: Commit gyb-generated files to master SwiftSyntax: Commit gyb-generated files to master Oct 23, 2019
@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - f33c95d

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - f33c95d

@ahoppen
Copy link
Member Author

ahoppen commented Oct 23, 2019

swiftlang/swift-syntax#159

@swift-ci Please smoke test

@ahoppen
Copy link
Member Author

ahoppen commented Oct 23, 2019

Just realized I hadn't updated the generated files to match. Let's restart the test.

swiftlang/swift-syntax#159

@swift-ci Please smoke test

@ahoppen
Copy link
Member Author

ahoppen commented Oct 24, 2019

Swift-CI didn’t pull in the SwiftSyntax sources. Try again.

swiftlang/swift-syntax#159

@swift-ci Please smoke test

@ahoppen
Copy link
Member Author

ahoppen commented Oct 24, 2019

Test failure seemed unrelated. Let’s try one more time.

swiftlang/swift-syntax#159

@swift-ci Please smoke test macOS platform

@ahoppen
Copy link
Member Author

ahoppen commented Oct 25, 2019

The bug causing CI tests to fail should be fixed now

swiftlang/swift-syntax#159

@swift-ci Please smoke test macOS platform

@ahoppen ahoppen merged commit d5c28d8 into swiftlang:master Oct 25, 2019
@ahoppen ahoppen deleted the forward-verify-generated-files branch October 25, 2019 18:34
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.

3 participants