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

Update lit-analyzer to fix ref and lit 2 directive issues #779

Merged
merged 7 commits into from
Jun 21, 2023

Conversation

florian-sanders-cc
Copy link
Contributor

@florian-sanders-cc florian-sanders-cc commented Jun 20, 2023

Fixes #679
Fixes #778

What does this PR do?

  • Updates lit-analyzer to 2.0.0-pre.3. I chose to stick with the specific version and not "^" because these are not stable releases. WDYT?
  • Removes the workaround for the ref directive.
  • Removes the globalAttributes workaround from tsconfig.json.

How to review

Run npm run components:check-lit => there should be 0 error.

I've already checked that the following components are still working correctly (focus handling mostly):

  • cc-orga-member-list,
  • cc-orga-member-card,
  • cc-ssh-key-list,
  • cc-tile-metrics
  • cc-pricing-estimation (newly merged).

Copy link
Contributor

@pdesoyres-cc pdesoyres-cc left a comment

Choose a reason for hiding this comment

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

LGTM

@florian-sanders-cc florian-sanders-cc force-pushed the lit-analyzer/update branch 2 times, most recently from 59e1add to 46edf84 Compare June 20, 2023 14:35
@github-actions
Copy link
Contributor

🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/lit-analyzer/update/index.html.

This preview will be deleted once this PR is closed.

@florian-sanders-cc florian-sanders-cc merged commit dd11487 into master Jun 21, 2023
@florian-sanders-cc florian-sanders-cc deleted the lit-analyzer/update branch June 21, 2023 09:15
@github-actions
Copy link
Contributor

🔎 The preview has been automatically deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants