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.

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

Update build process, use GitHub workflows #127

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

nyanhp
Copy link
Member

@nyanhp nyanhp commented Mar 19, 2024

Description

I did not test the publishing step, but the validation seemed to work. The module works as well, but it is still a bit like open heart surgery insofar as that we don't have integration tests :)

New: Develop branch gets pre-released on every commit (i.e. after pull requests), to release to prod:

  • git switch develop
  • git pull
  • Update-ModuleManifest AutomatedLab.Common/AutomatedLab.Common.psd1 -ModuleVersion <Next.Version.Number>
  • git add AutomatedLab.Common/AutomatedLab.Common.psd1
  • git commit -m "Release prepared"
  • git checkout master
  • git pull
  • git merge --squash develop
  • git push
  • git switch develop
  • git push

I hope that process is bearable @raandree, otherwise I'm open to ideas :)

  • - I have tested my changes.
  • - I have updated CHANGELOG.md and added my change to the Unreleased section
  • - The PR has a meaningful title.
  • - I updated my fork/branch and have integrated all changes from AutomatedLab.common/develop before creating the PR.

Type of change

  • Bug fix
  • New functionality
  • Breaking change
  • Documentation

How was the change tested?

🤡
Local builds, builds in my fork, did not test all cmdlets

@nyanhp nyanhp requested a review from raandree March 19, 2024 16:50
@nyanhp nyanhp force-pushed the feature/shinynewbuild branch from 7f547d6 to 1d102fc Compare March 30, 2024 11:35
@nyanhp
Copy link
Member Author

nyanhp commented Apr 26, 2024

@raandree any opinions?

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.

1 participant