Skip to content

Get ready to make public#11

Merged
amarsan merged 2 commits intomainfrom
anne/GetReadyForPublic
Jan 5, 2026
Merged

Get ready to make public#11
amarsan merged 2 commits intomainfrom
anne/GetReadyForPublic

Conversation

@amarsan
Copy link
Contributor

@amarsan amarsan commented Jan 5, 2026

This repo is under change control. All changes must be made in a branch from main and reviewed.

Description of Changes

This updates our workflows in preparation for going public.

  • Removed the CE proget configuration from the cicd workflow because it is not needed
  • Added the step to publish to npmjs.com to the publish workflow.

Security

REMINDER: All file contents are public.

  • I have ensured no secure credentials or sensitive information remain in code, metadata, comments, etc. Of particular note:
    No temporary testing changes committed such as API base URLs, access tokens, print/log statements, etc.
  • My changes do not introduce any security risks, or any such risks have been properly mitigated.

Describe briefly what security risks you considered, why they don't apply, or how they've been mitigated.

Reviewers

  • I have assigned the appropriate reviewer(s).

Minimally, a second set of eyes is needed ensure no non-public information is published. Consider also including subject-matter experts and privacy/security reviewers, if appropriate.

@amarsan amarsan requested a review from scottfavre January 5, 2026 17:22
@amarsan
Copy link
Contributor Author

amarsan commented Jan 5, 2026

I checked the cicd action, it ran successfully.

@amarsan amarsan merged commit 5c27325 into main Jan 5, 2026
1 check passed
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