Skip to content

Commit dcdb182

Browse files
authored
Merge pull request #940 from microsoftgraph/chore/remove-publish-npm-step
removing publish step since azure is now in charge of that
2 parents 6348979 + d346cae commit dcdb182

2 files changed

Lines changed: 2 additions & 29 deletions

File tree

.azure-pipelines/release.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,7 @@ extends:
5858
- script: npm run build
5959
displayName: 'Build project'
6060
workingDirectory: '$(Build.SourcesDirectory)'
61-
62-
# excluding test for now since it requires a tenant and secrets to run
63-
# TODO: create secrets to the pipeline similar to https://github.com/microsoftgraph/msgraph-sdk-typescript/settings/secrets/actions and the build.yml file
64-
# - script: npm run test
65-
# displayName: 'Run tests'
66-
# workingDirectory: '$(Build.SourcesDirectory)'
67-
61+
6862
- task: PowerShell@2
6963
displayName: 'Copy README.md to packages'
7064
inputs:

.github/workflows/build.yml

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -62,25 +62,4 @@ jobs:
6262
run: exit 0
6363
- name: One or more build matrix options failed
6464
if: ${{ contains(needs.*.result, 'failure') }}
65-
run: exit 1
66-
67-
publish-npm:
68-
if: startsWith(github.ref, 'refs/tags/')
69-
needs: build
70-
environment:
71-
name: production_feed
72-
runs-on: ubuntu-latest
73-
steps:
74-
- uses: actions/checkout@v5
75-
- uses: actions/setup-node@v6
76-
with:
77-
node-version: 22
78-
registry-url: https://registry.npmjs.org/
79-
- name: Copy README.md to packages
80-
shell: pwsh
81-
run: ./scripts/copy-readme.ps1
82-
- run: npm ci
83-
- run: npm run build
84-
- run: npx lerna publish from-package --no-push --yes
85-
env:
86-
NODE_AUTH_TOKEN: ${{secrets.npm_token}}
65+
run: exit 1

0 commit comments

Comments
 (0)