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.

Sign up for GitHub

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

Jump to bottom

Dry-Run Module Publishing in CI #265

Merged
merged 8 commits into from
Sep 25, 2019
Merged

Dry-Run Module Publishing in CI #265

merged 8 commits into from
Sep 25, 2019

Conversation

vexx32
Copy link
Owner

@vexx32 vexx32 commented Sep 25, 2019 •
edited

PR Summary

Add a job to dry-run the publish steps by publishing to a file system location and then publishing the resulting nupkg as an artifact.

Context

Changes

  • Add new job to yaml
  • Modify scripts to accommodate

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

@vexx32 vexx32 added PR-Needs-Review 🔍 Let's take a closer look! Category-Build 🚀 Modifies the build / deployment system labels Sep 25, 2019
@vexx32 vexx32 force-pushed the Build/TestPublish branch 2 times, most recently from b6846a2 to 9a483fa Compare September 25, 2019 18:12
@vexx32
🚀 Fix nupkg path (hopefully)
feb06d5 
Add debug info to deployment step
@vexx32 vexx32 merged commit 9d046f5 into master Sep 25, 2019
@vexx32 vexx32 deleted the Build/TestPublish branch September 25, 2019 19:36
@vexx32 vexx32 removed the PR-Needs-Review 🔍 Let's take a closer look! label Sep 25, 2019
thomasrayner pushed a commit to thomasrayner/PSKoans that referenced this pull request Oct 23, 2019
@vexx32 @thomasrayner
🚀 Dry-Run Module Publishing in CI (vexx32#265)
f51bc82 
* 🚀 Add test-publish job

* 🚀 publish built module regardless

* 🚀 Change artifact name

* 🚀 ensure environment is setup

* 🚀 Publish dependencies to filesystem

* 🚀 Fix nupkg path (hopefully)
Add debug info to deployment step

* 🚀 Adjust artifact names

* 📝 Adjust namings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steviecoaster steviecoaster Awaiting requested review from steviecoaster

Assignees
No one assigned
Labels
Category-Build 🚀 Modifies the build / deployment system
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@vexx32