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

Some CI improvements #190

Merged
merged 4 commits into from
Feb 4, 2020
Merged

Some CI improvements #190

merged 4 commits into from
Feb 4, 2020

Conversation

gabro
Copy link
Member

@gabro gabro commented Feb 4, 2020

  • fix prettier check
  • remove travis badge from README
  • add publishing of VSIX to the latest release (the idea is that when the release workflow runs it was triggered by a release)

@@ -22,7 +22,8 @@ jobs:
id: get_version
run: echo ::set-output name=NEW_VERSION::${GITHUB_REF#/refs/tags/v}
- run: yarn install
- run: yarn version --no-git-tag-version --new-version ${{
- run:
Copy link
Member

Choose a reason for hiding this comment

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

This is needed to fix the interaction with prettier

Suggested change
- run:
- run: |

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, fixed

env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
upload_url: { { steps.get_upload_url.outputs.UPLOAD_URL } }
Copy link
Member

Choose a reason for hiding this comment

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

Woah, this is cool!

Copy link
Member Author

Choose a reason for hiding this comment

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

If it works, sure 😅

@gabro
Copy link
Member Author

gabro commented Feb 4, 2020

@olafurpg what do you think about triggering a release to see whether this works?

@gabro gabro merged commit 614991d into scalameta:master Feb 4, 2020
@gabro gabro deleted the fix-ci branch February 4, 2020 12:00
@olafurpg
Copy link
Member

olafurpg commented Feb 4, 2020

@gabro sounds good! I think we have a few fixes in master that would be good to release as well

kasiaMarek pushed a commit to kasiaMarek/metals-vscode that referenced this pull request Mar 29, 2023
…rn/types/node-14.11.8

Bump @types/node from 14.11.5 to 14.11.8
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