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

General Project Questions #4

Open
dahlbyk opened this issue Jun 18, 2018 · 24 comments
Open

General Project Questions #4

dahlbyk opened this issue Jun 18, 2018 · 24 comments

Comments

@dahlbyk
Copy link
Owner

dahlbyk commented Jun 18, 2018

  1. Should this module be distributed via Chocolatey, or only PS Gallery?
  2. Following on from Module no-longer loads after #376 (manifest moved to src/ directory). posh-git#467, my recommendation would be to have the psd1 and psm1 live in the project root instead of inside src, so the module can be loaded easily from a fresh clone.
  3. Since posh-git v0.x will continue to support older PowerShell, should this module target Windows PowerShell 5.1 and PowerShell Core 6+?
@JeremySkinner
Copy link
Collaborator

Should this module be distributed via Chocolatey, or only PS Gallery?

What’re the download stats on ps gallery vs choclatey? My initial thought was ps gallery only, but if posh-git gets lots of downloads from chocately maybe we should distribute there too.

Following on from dahlbyk/posh-git#467, my recommendation would be to have the psd1 and psm1 live in the project root instead of inside src

Good idea

should this module target Windows PowerShell 5.1 and PowerShell Core 6+

Yes I think so.

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 18, 2018

What’re the download stats on ps gallery vs choclatey? My initial thought was ps gallery only, but if posh-git gets lots of downloads from chocately maybe we should distribute there too.

Downloads for 0.7.3 are about 25k/15k in favor of Chocolatey. Of course, if it weren't an option it's unclear how many would shift to PS Gallery.

@JeremySkinner
Copy link
Collaborator

Let’s keep it on chocately too then. Do you want to be responsible for managing releases and pushing packages or would you like me to handle that too?

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 19, 2018

Let's share!

@JeremySkinner
Copy link
Collaborator

Perfect!

@JeremySkinner
Copy link
Collaborator

I've changed my mind about Chocolatey...I think we should only publish on PS gallery initially, and we can always add a Chocolatey package later if it's frequently requested.

@JeremySkinner
Copy link
Collaborator

@dahlbyk There's already a posh-ssh module on powershell gallery (https://www.powershellgallery.com/packages/posh-ssh/2.0.2). Do we want to change the project name?

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 19, 2018

There's already a posh-ssh module on powershell gallery

Oh no! I'm fond of posh-sshit (pronounced "posh-ssh-it", of course).

posh-ssh-agent would work for current scope, but if the scope is going to expand to handle tab expansion for ssh... Maybe posh-sshell? Open to ideas, but yeah a rename is in order.

@JeremySkinner
Copy link
Collaborator

I like posh-sshell, I think some people might not like posh-sshit!

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 19, 2018

I like posh-sshell

Done

@JeremySkinner
Copy link
Collaborator

Thanks!

@JeremySkinner
Copy link
Collaborator

I've updated all text references and created an appveyor build

@JeremySkinner
Copy link
Collaborator

@dahlbyk Please could you turn on protected branches for master (if it's not on already)?

@JeremySkinner
Copy link
Collaborator

JeremySkinner commented Jun 22, 2018

@dahlbyk thanks for enabling that, could you set it to allow me to push still though? I mainly wanted it to stop accidental force pushes. Thanks 😄

Edit: I also can't merge branches for PRs as It's also not triggering builds for branches (even though that's now enabled in the yml). I'm wondering if it's because the build is under my account rather than yours? Could you check the webhook settings as I don't have access to that?

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 24, 2018

Sorry! Webhook is definitely not set up - can you DM me the link? Or can you add me to the Appveyor project (dahlbyk@gmail)?

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 24, 2018

(Also, I have disabled requiring status checks for now)

@JeremySkinner
Copy link
Collaborator

Thanks, I've sent you an invite to the appveyor project, let me know if you don't get it.

@JeremySkinner
Copy link
Collaborator

Doesn't look like the status checks have been disabled...I'm still seeing "Required statuses must pass before merging" on the PR page, from the terminal I get ! [remote rejected] master -> master (protected branch hook declined)

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 25, 2018

Yeah, I must not have hit Save. Fixed.

@JeremySkinner
Copy link
Collaborator

@dahlbyk Everything should now be in place for feature parity with posh-git 0.x. I’ve also started working on new features in the feature/config-parser branch.

Anything else you’d like to see in place before pushing 0.1 to ps gallery?

@dahlbyk
Copy link
Owner Author

dahlbyk commented Jun 25, 2018

Check out #10

As for the actual release process, my process for posh-git is typically:

  1. Create commit on release-vX.X.X that updates psd1 links and drops Prerelease (dahlbyk/posh-git@9c34bb9)
  2. Create annotated vX.X.X tag for that commit
  3. Create another commit that bumps version number and restores Prerelease to something we'll never actually release (so we can tell the difference between errors from clones vs published versions)
  4. Push branch + tag to GitHub and open PR

Anything you'd like to do differently here?

@JeremySkinner
Copy link
Collaborator

No that sounds like a good approach, I'm happy to stick with the same workflow.

@JeremySkinner
Copy link
Collaborator

I've written a script to automate the release tagging process - https://github.com/dahlbyk/posh-sshell/blob/master/prepare-release.ps1.

There's also a publish.ps1 that handles publishing to gallery.

Feel free to use either in posh-git if you think they'd be useful.

@JeremySkinner
Copy link
Collaborator

I've pushed out 0.1 to PS Gallery and added you as an owner

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

No branches or pull requests

2 participants