-
Notifications
You must be signed in to change notification settings - Fork 156
DX-809 update to use preferred OIDC for npm publish in github actions #1333
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
Open
lloyd-cio
wants to merge
86
commits into
segmentio:master
Choose a base branch
from
customerio:DX-809
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Added Test Cases for browser check * checked with pwd * changed location * fixed the location * added logging * renamed the file locaiton * added more logging * removed node modules * debug logs * update node modules * removed tsc * added npm i * added debug artifact * updated artifact * more logging * added more loggin * added more tar logs * fixed test * updated test cases * Added Tsub as vendor * fixed vendor * Revert "fixed vendor" This reverts commit 8f50ad6. * Revert "Added Tsub as vendor" This reverts commit 93448b5. * Revert "updated test cases" This reverts commit 12eccf2. * reverted and update node_modules * Reapply "updated test cases" This reverts commit 08d5fbd. * Reapply "Added Tsub as vendor" This reverts commit d1b821d. * Reapply "fixed vendor" This reverts commit 5b32ad6. * removed dist folder * added comments
Part of INAPP-13292 Related PR: customerio/gist-web#82
* Add Start of a Mise Config It's due * code review feedback * Prevent teardown error in test * Fix cache step From https://github.com/actions/setup-node/blob/main/docs/advanced-usage.md#caching-packages-data To fix this: Post job cleanup. Error: Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved.
Part of INAPP-13292
* Uses remotely injected configuration if present * Updated to use latest version of gist-web --------- Co-authored-by: Bernard Gatt <[email protected]>
Co-authored-by: Bernard Gatt <[email protected]>
* updated dependecies * remove mise and update package * added Mise comments
* Update in-app plugin * Updated with polyfill fix --------- Co-authored-by: Bernard Gatt <[email protected]>
Co-authored-by: Bernard Gatt <[email protected]>
* - Injecting Gist library from CDN - Removed NPM dependency for gist web - Added update_cdp script * Fixes tests * Updated browser packages script * Removed whitespace * Added configurable version parameter --------- Co-authored-by: Bernard Gatt <[email protected]>
… latest gist-web version. (#63) Co-authored-by: Bernard Gatt <[email protected]>
* Fixes linting problems * Added setAnonymousId Updated to latest Gist library Added tests --------- Co-authored-by: Bernard Gatt <[email protected]>
* Support overriding the messageId in the node SDK * Mise tasks to improve local DX Co-authored-by: sydneycollins-cio <[email protected]>
🦋 Changeset detectedLatest commit: ac0c9ad The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
yarn changeset. Read about changesets here).