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

Project created with a new nightly package is requesting an outdated engine #6560

Closed
1 of 2 tasks
sylwiabr opened this issue May 4, 2023 · 2 comments · Fixed by #6570
Closed
1 of 2 tasks

Project created with a new nightly package is requesting an outdated engine #6560

sylwiabr opened this issue May 4, 2023 · 2 comments · Fixed by #6570
Assignees
Labels
--bug Type: bug --regression Important: regression p-high Should be completed in the next sprint

Comments

@sylwiabr
Copy link
Member

sylwiabr commented May 4, 2023

Discord username

No response

What type of issue is this?

Permanent – Occurring repeatably

Is this issue blocking you from using Enso?

  • Yes, I can't use Enso because of this issue.

Is this a regression?

  • Yes, previous version of Enso did not have this issue.

What issue are you facing?

IDE is reporting unsupported Engine version for a project just created with newest nightly.

Expected behaviour

No warning - I have newer version then the one required

How we can reproduce it?

No response

Screenshots or screencasts

image

Enso Version

nightly 04.05

Browser or standalone distribution

Standalone distribution

Browser Version or standalone distribution

standalone

Operating System

MacOS

Operating System Version

No response

Hardware you are using

No response

@sylwiabr sylwiabr added --bug Type: bug triage labels May 4, 2023
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board May 4, 2023
@sylwiabr sylwiabr added --regression Important: regression p-highest Should be completed ASAP p-high Should be completed in the next sprint and removed p-highest Should be completed ASAP labels May 4, 2023
@hubertp
Copy link
Collaborator

hubertp commented May 4, 2023

I don't believe this is a regression. It has been like that for a while if not from the inception of that version check.

@sylwiabr
Copy link
Member Author

sylwiabr commented May 4, 2023

The nightly package from yesterday does not have this issue so it is a regression. It is like that for dev builds where you have dev engine version but for nightlies it was fixed.

@4e6 4e6 removed the triage label May 4, 2023
@mwu-tow mwu-tow moved this from ❓New to 🔧 Implementation in Issues Board May 4, 2023
@vitvakatu vitvakatu moved this from 🔧 Implementation to 👁️ Code review in Issues Board May 8, 2023
@mergify mergify bot closed this as completed in #6570 May 8, 2023
mergify bot pushed a commit that referenced this issue May 8, 2023
This PR fixes #6560.

The fix has a few elements:
1) Bumps the Engine requirement to the latest release, namely `2023.1.1`.
2) Changed the logic of checking whether a given version matches the requirement. Previously, we relied on `VersionReq` from `semver` crate which did not behave intuitively when the required version had a prerelease suffix. Now we rely directly on Semantic Versioning rules of precedence.
3) Code cleanups, including deduplicating 3 copies of the version-checking code, and moving some tests to more sensible places.
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--bug Type: bug --regression Important: regression p-high Should be completed in the next sprint
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants