Skip to content

feat: Deprecate support for all macOS versions below High Sierra #3588

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 1 commit into from
May 17, 2018

Conversation

rosen-vladimirov
Copy link
Contributor

Add deprecation message in case the current macOS version is below High Sierra. As the latest Xcode versions can be installed only on this OS and we state that we support only latest Xcode, we have to deprecate the support for other OS versions.
The deprecation message will be shown on each CLI command.
As the deprecation messages from CLI should be shown in Sidekick as well, expose new methods in CLI's sysInfo moodule, so Sidekick can get the warnings and show them.
In order to get the warnings for Node.js version, change the way verifyNodeVersion is called.
Expose method for getting the currently supported Node.js version range.
Add tests for the new methods.

PR Checklist

What is the current behavior?

All macOS versions are supported

What is the new behavior?

When macOS is not HighSierra, CLI prints a warning that this version will not be supported in the future.

Implements #3570

@rosen-vladimirov rosen-vladimirov added this to the 4.1.0 milestone May 15, 2018
@rosen-vladimirov rosen-vladimirov self-assigned this May 15, 2018
@rosen-vladimirov rosen-vladimirov requested a review from Fatme May 15, 2018 19:34
PublicAPI.md Outdated
* Definition
```TypeScript
/**
* Gets all globall warnings for the current environment, for example Node.js version compatibility, OS compatibility, etc.
Copy link
Contributor

Choose a reason for hiding this comment

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

globall -> global

Add deprecation message in case the current macOS version is below High Sierra. As the latest Xcode versions can be installed only on this OS and we state that we support only latest Xcode, we have to deprecate the support for other OS versions.
The deprecation message will be shown on each CLI command.
As the deprecation messages from CLI should be shown in Sidekick as well, expose new methods in CLI's `sysInfo` moodule, so Sidekick can get the warnings and show them.
In order to get the warnings for Node.js version, change the way `verifyNodeVersion` is called.
Expose method for getting the currently supported Node.js version range.
Add tests for the new methods.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/deprecate-macos-sierra branch from b3595a7 to 64bab09 Compare May 17, 2018 09:14
@rosen-vladimirov rosen-vladimirov merged commit f7a4594 into master May 17, 2018
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/deprecate-macos-sierra branch May 17, 2018 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants