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

bump dependencies & simplify enabled rules logic #98

Closed
wants to merge 3 commits into from

Conversation

43081j
Copy link
Contributor

@43081j 43081j commented Apr 29, 2020

do it only in one place and bump some deps.

seems to work fine, let me know if anything looks off.

typescript can't be updated until WCA has the same version bump.

edit: no clue why ci fails, exact same stuff passes locally with a clean repo (git clean -xdf)

@43081j
Copy link
Contributor Author

43081j commented May 7, 2020

closing because 1.2.0 shuffles a whole load of this logic around

@43081j 43081j closed this May 7, 2020
@runem
Copy link
Owner

runem commented May 7, 2020

I was actually just writing a comment in here which I will post/add nevertheless:

The reason the CI fails is because of problems with vsce and mono-repositories. Unfortunately, vsce doesn't yet support mono-repositories with linked dependencies. You can read more about the issue here: microsoft/vscode-vsce#203. In order to get around this problem and debug locally developed dependencies (ts-lit-plugin and lit-analyzer) in the vscode extension project, I had to make a small script that copies locally built lib files to packages/vscode-lit-plugin/node_modules/{ts-lit-plugin, lit-analyzer} (see copylink script). The problem we experience in this PR occurs because lit-analyzer has updated dependencies, but the dependencies in packages/vscode-lit-plugin are installed using dependencies of the published version of lit-analyzer (thus installing 'old' dependencies). vsce checks for out-of-sync dependencies when packaging and crashes. I found a way around this on the 1.2.0 branch, and I can cherry-pick the fix if necessary.

@runem
Copy link
Owner

runem commented May 7, 2020

In general if we change code around rule logic I would love it to be based on the 1.2.0 branch, so we avoid too many merge conflicts 👍 But if you think this (or another PR) is very important for an upcoming release of 1.1.x I'm also fine with handling the merge conflicts :-)

@43081j
Copy link
Contributor Author

43081j commented May 7, 2020

its ok thats a shame about the vscode issue though.

ill probably throw together a different branch on top of the 1.2 branch, if anything. i think your new rule sets achieve what i was going for mostly. the dependency updates are nice to have though so i might still do that, as the newer ava has debugging built in (like node's --inspect).

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