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

refactor: initialisation of feature flag status [IDE-346] #458

Merged
merged 2 commits into from
Jun 14, 2024

Conversation

teodora-sandu
Copy link
Contributor

@teodora-sandu teodora-sandu commented May 20, 2024

Description

Refactors the logic used for initialising the feature flag status at startup.

To test, run the VSCode sandbox then:

  • configure the organisation used in Snyk Settings to be one without snykCodeConsistentIgnores
  • run a scan (Code at least) and open a vulnerability to see the existing suggestion panel
  • restart VSCode
  • configure the organisatio nused in Snyk Settings to be one with snykCodeConsistentIgnores (e.g. ide-consistent-ignores-test)
  • run a scan (Code at least) and open a vulnerability to see the new suggestion panel

Note! We are aware of a bug where changing the organisation in Snyk Settings does not trigger an update of the organisation in LS and so the feature flag status doesn't change because the org hasn't, as far as the command in LS is aware.

Checklist

  • Tests added and all succeed
  • Linted
  • CHANGELOG.md updated
  • README.md updated, if user-facing

@cat2608 cat2608 changed the title fix: initialisation of feature flag status fix: initialisation of feature flag status [IDE-346] May 22, 2024
@teodora-sandu teodora-sandu force-pushed the fix/feature-flag-status-init branch 2 times, most recently from 894d391 to d25dbc9 Compare May 22, 2024 16:39
@teodora-sandu teodora-sandu changed the title fix: initialisation of feature flag status [IDE-346] refactor: initialisation of feature flag status [IDE-346] May 22, 2024
@teodora-sandu teodora-sandu marked this pull request as ready for review May 23, 2024 08:19
@teodora-sandu teodora-sandu requested a review from a team as a code owner May 23, 2024 08:19
@teodora-sandu teodora-sandu force-pushed the fix/feature-flag-status-init branch 3 times, most recently from 14ae1a5 to a4c9442 Compare June 13, 2024 16:18
@@ -1,10 +1,9 @@
# Snyk Security Changelog

## [2.12.2]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't need a new version for each PR, just need to add changelog messages to the next version after the latest release

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, my fault...

Copy link
Contributor

@bastiandoetsch bastiandoetsch left a comment

Choose a reason for hiding this comment

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

LGTM, please rebase against main to get the latest protocol version - you'll need to update the changelog, unfortunately, again :(

@teodora-sandu teodora-sandu merged commit 248af1e into main Jun 14, 2024
7 checks passed
@teodora-sandu teodora-sandu deleted the fix/feature-flag-status-init branch June 14, 2024 08:27
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.

2 participants