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.

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

Revert "Standardize the entry-point for Julia execution (#50974)" #51196

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

lgoettgens
Copy link
Contributor

This reverts commit 8e14322.

As proposed by @Seelengrab in #50974 (comment).

This is not meant to be a permanent revert of #50974, but a possibility to have working PkgEval and nightly testing for packages (https://github.com/julia-actions/julia-buildpkg @latest doesn't work with it) until a solution to the discussion there has been decided on.

@vtjnash
Copy link
Member

vtjnash commented Sep 5, 2023

As far as I can tell from the current discussion, this is our best current option. It avoid making the final decision on what API will be used unnecessarily stressful in the short-term, allows people to voice their opinions, and then possibly we will still just re-land this PR by the end of the week, if that is the decision. @Keno

Copy link
Member

@Keno Keno left a comment

Choose a reason for hiding this comment

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

Per discussion earlier today, this is accepted for the time being, with apologies to the people who already updated their scripts and a goal to get this brought back in the next few days with one of the options to smooth the transition.

@DilumAluthge DilumAluthge added the revert This reverts a previously merged PR. label Sep 5, 2023
@Keno Keno merged commit ba16663 into JuliaLang:master Sep 5, 2023
@lgoettgens lgoettgens deleted the revert-50974-kf/mainentry branch September 5, 2023 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
revert This reverts a previously merged PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants