Skip to content

Commit

Permalink
Merge pull request #24 from joshmu/fix_ripgrep-compatibility
Browse files Browse the repository at this point in the history
fix: allow new rg install to define correct platform binary
  • Loading branch information
joshmu authored Apr 1, 2024
2 parents 487dbf5 + 696cec7 commit cf38ed4
Showing 1 changed file with 10 additions and 13 deletions.
23 changes: 10 additions & 13 deletions .github/workflows/publish-vscode-extension.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,26 +15,23 @@ jobs:
node-version: 20
- run: npm ci

# Attempted to set executable permissions for ripgrep, but it didn't work
# - name: Set Executable Permissions for ripgrep
# run: chmod +x ./node_modules/@vscode/ripgrep/bin/rg
- name: Remove ripgrep bin folder, to allow consumers to redefine their OS specific binaries
run: rm -rf ./node_modules/@vscode/ripgrep/bin

- name: Semantic Release
run: npx semantic-release
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# Add any other tokens or configuration required by semantic-release plugins

# Automated pipeline environment causes a permssion issue with the ripgrep binary in the published extension...
# for now I'll need to continue to publish the package locally for it to work correctly
# - name: Install Visual Studio Marketplace CLI
# run: npm install -g vsce
- name: Install Visual Studio Marketplace CLI
run: npm install -g vsce

# - name: Package & Publish Extension to Visual Studio Marketplace
# run: vsce publish --pat ${{ secrets.VS_MARKETPLACE_TOKEN }}
- name: Package & Publish Extension to Visual Studio Marketplace
run: vsce publish --pat ${{ secrets.VS_MARKETPLACE_TOKEN }}

# - name: Install Open VSX CLI
# run: npm install -g ovsx
- name: Install Open VSX CLI
run: npm install -g ovsx

# - name: Package & Publish Extension to Open VSX Registry
# run: ovsx publish --pat ${{ secrets.OPEN_VSX_TOKEN }}
- name: Package & Publish Extension to Open VSX Registry
run: ovsx publish --pat ${{ secrets.OPEN_VSX_TOKEN }}

0 comments on commit cf38ed4

Please sign in to comment.