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

Store .vsix as part of appveyor CI #322

Merged
merged 4 commits into from Feb 25, 2018
Merged

Conversation

vazexqi
Copy link
Contributor

@vazexqi vazexqi commented Feb 25, 2018

What does this PR do?

Stores the .vsix as part of the artifact for debugging purposes.

What issues does this PR fix or reference?

@W-4700676@

@vazexqi
Copy link
Contributor Author

vazexqi commented Feb 25, 2018

There seems to be a build issue that just surfaced. It looks to be related to npm/npm#19877

@codecov
Copy link

codecov bot commented Feb 25, 2018

Codecov Report

Merging #322 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #322   +/-   ##
========================================
  Coverage    77.73%   77.73%           
========================================
  Files          127      127           
  Lines         4856     4856           
  Branches       780      780           
========================================
  Hits          3775     3775           
  Misses         888      888           
  Partials       193      193

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40bbf5e...f6dc6b1. Read the comment docs.

@vazexqi
Copy link
Contributor Author

vazexqi commented Feb 25, 2018

I followed what they have done at angular/angular-cli#9713 to fix this issue temporarily - add the dependency to a specific version of ajv at the top-level.

@vazexqi
Copy link
Contributor Author

vazexqi commented Feb 25, 2018

@guw - Artifacts are now stored and available at https://circleci.com/gh/forcedotcom/salesforcedx-vscode/218#artifacts/containers/0

@ruthemmanuelle - Just FYI, this is also a way for you to generate the .vsix yourself for installation. Just open a PR against any branch and wait for the Circle CI just to finish (no need to wait for Travis or AppVeyor) and you should be able to get the .vsix that you want.

CC: @DatGuyJonathan @lcampos - I also needed to fix the ajv issue that just popped up (not due to any of our changes). But you should be able to publish this Thursday with no issues.

@vazexqi
Copy link
Contributor Author

vazexqi commented Feb 25, 2018

As part of this work item, I realized that we don't have a proper .vscodeignore for the salesforcedx-vscode-apex-debugger and salesforcedx-vscode-lightning. This is why the size of the .vsix are larger as seen in https://ci.appveyor.com/project/guw/salesforcedx-vscode/build/develop-2008/artifacts

This happens because as part of the CI, we run the tests which will bring .vscode-test which is a binary file. This doesn't happen when we do the publishing because we use a different run (more similar to what we do in CircleCI today) which doesn't bring in the .vscode-test.

However, we can avoid this size expansion issue altogether by providing a .vscodeignore. I'll do that as part of another PR.

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.

None yet

1 participant