Skip to content

improve example created by package init --type executable #4122

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 4 commits into from
Feb 12, 2022

Conversation

tomerd
Copy link
Contributor

@tomerd tomerd commented Feb 11, 2022

motivation: example is out of date with best practices

changes:

  • use @main istead of main.swift
  • test executable directly instead of shelling out to a process

rdar://82119401

@tomerd
Copy link
Contributor Author

tomerd commented Feb 11, 2022

@swift-ci please smoke test

@tomerd tomerd added the ready Author believes the PR is ready to be merged & any feedback has been addressed label Feb 11, 2022
@neonichu
Copy link
Contributor

Looks like maybe what we're doing here isn't supported in the older toolchain used for self-hosted tests? The error messages are a bit confusing to me.

@tomerd
Copy link
Contributor Author

tomerd commented Feb 11, 2022

@swift-ci please smoke test

1 similar comment
@tomerd
Copy link
Contributor Author

tomerd commented Feb 11, 2022

@swift-ci please smoke test

motivation: example is out of date with best practices

changes:
* use @main istead of main.swift
* test executable directly instead of shelling out to a process
* adjust tests

rdar://82119401
@tomerd
Copy link
Contributor Author

tomerd commented Feb 12, 2022

@swift-ci please smoke test

@tomerd
Copy link
Contributor Author

tomerd commented Feb 12, 2022

@swift-ci please smoke test

@tomerd tomerd merged commit 85a84d9 into swiftlang:main Feb 12, 2022
shahmishal added a commit that referenced this pull request Feb 12, 2022
shahmishal added a commit that referenced this pull request Feb 12, 2022
tomerd added a commit that referenced this pull request Feb 12, 2022
tomerd added a commit that referenced this pull request Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready Author believes the PR is ready to be merged & any feedback has been addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants