Prevent branches from being merged that have a `-dev` suffix on the version number

Firepup650 commented

2024-04-18 13:43:58 +00:00

Owner

I'm not sure how/if this is possible, but it would be a nice sanity check.

Firepup650 added the

Kind/Enhancement

Priority

Low

labels 2024-04-18 13:44:13 +00:00

Firepup650 commented

2024-04-18 13:44:56 +00:00

Author

Owner

@9pfs any ideas on this?

9pfs commented

2024-04-18 14:45:01 +00:00

Collaborator

If you disable pushes for the main branch and only do PRs, maybe you could use forgejo actions

Firepup650 commented

2024-04-18 17:16:12 +00:00

Author

Owner

I mean I don't exactly want to do that, I mean if something is PR'd, don't allow it to be merged unless it doesn't contain -dev in the version

Essentially:

I should still be able to push as I need to, regardless
Any PR's should be blocked from merging if they're a -dev version

I mean I don't exactly want to do that, I mean if something is PR'd, don't allow it to be merged unless it doesn't contain `-dev` in the version Essentially: 1. I should still be able to push as I need to, regardless 2. Any PR's should be blocked from merging if they're a `-dev` version

Firepup650 commented

2024-04-18 17:17:42 +00:00

Author

Owner

Also I should have people update version numbers, I guess

Firepup650 changed title from ~~Prevent branches from being merged that add a -dev suffix to the version number~~ to Prevent branches from being merged that have a -dev suffix on the version number

2024-04-18 17:26:21 +00:00

9pfs commented

2024-04-18 23:37:44 +00:00

Collaborator

I mean I don't exactly want to do that, I mean if something is PR'd, don't allow it to be merged unless it doesn't contain -dev in the version

Essentially:

I should still be able to push as I need to, regardless

Any PR's should be blocked from merging if they're a -dev version

Could you just use forgejo actions then?

> I mean I don't exactly want to do that, I mean if something is PR'd, don't allow it to be merged unless it doesn't contain `-dev` in the version > > Essentially: > 1. I should still be able to push as I need to, regardless > 2. Any PR's should be blocked from merging if they're a `-dev` version > Could you just use forgejo actions then?

Firepup650 commented

2024-04-19 17:39:31 +00:00

Author

Owner

idk how to do that @9pfs, could you set one up?

9pfs commented

2024-04-20 01:31:26 +00:00

Collaborator

idk how to do that @9pfs, could you set one up?

Where would the forgejo actions runner be running?

> idk how to do that @9pfs, could you set one up? Where would the forgejo actions runner be running?

Firepup650 commented

2024-04-20 01:32:55 +00:00

Author

Owner

Where would the forgejo actions runner be running?

Huh? What do you mean?

> Where would the forgejo actions runner be running? Huh? What do you mean?

9pfs commented

2024-04-20 01:33:38 +00:00

Collaborator

Where would the forgejo actions runner be running?

Huh? What do you mean?

You'd have to have a forgejo actions runner somewhere to run workflows

> > Where would the forgejo actions runner be running? > > Huh? What do you mean? You'd have to have a forgejo actions runner somewhere to run workflows

Firepup650 commented

2024-04-20 01:34:49 +00:00

Author

Owner

You'd have to have a forgejo actions runner somewhere to run workflows

Uh. idk what that would entail exactly, idek what that is

> You'd have to have a forgejo actions runner somewhere to run workflows Uh. idk what that would entail exactly, idek what that is

9pfs commented

2024-04-20 01:38:21 +00:00

Collaborator

You'd have to have a forgejo actions runner somewhere to run workflows

Uh. idk what that would entail exactly, idek what that is

https://forgejo.org/docs/latest/user/actions/

> > You'd have to have a forgejo actions runner somewhere to run workflows > > Uh. idk what that would entail exactly, idek what that is https://forgejo.org/docs/latest/user/actions/

Firepup650 commented

2024-04-20 01:43:16 +00:00

Author

Owner

https://forgejo.org/docs/latest/user/actions/

That's a large document.

> https://forgejo.org/docs/latest/user/actions/ That's a large document.

9pfs commented

2024-04-20 01:51:14 +00:00

Collaborator

https://forgejo.org/docs/latest/user/actions/

That's a large document.

suggests trying the quick start

> > https://forgejo.org/docs/latest/user/actions/ > > That's a large document. _suggests trying the quick start_

Firepup650 commented

2024-04-20 01:55:19 +00:00

Author

Owner

suggests trying the quick start

That doesn't even mention how to set up a runner thingy, just how to do a demo workflow

> _suggests trying the quick start_ That doesn't even mention how to set up a runner thingy, just how to do a demo workflow

9pfs commented

2024-04-20 02:03:37 +00:00

Collaborator

suggests trying the quick start

That doesn't even mention how to set up a runner thingy, just how to do a demo workflow

https://forgejo.org/docs/latest/admin/actions/#forgejo-runner

> > _suggests trying the quick start_ > > That doesn't even mention how to set up a runner thingy, just how to do a demo workflow https://forgejo.org/docs/latest/admin/actions/#forgejo-runner

Firepup650 commented

2024-04-20 05:34:14 +00:00

Author

Owner

https://forgejo.org/docs/latest/admin/actions/#forgejo-runner

I mean I wouldn't run that on fpi for fear of it burning a lot of my limited CPU, could you get instructions so I could set it up on nest when I have time?

> https://forgejo.org/docs/latest/admin/actions/#forgejo-runner I mean I wouldn't run that on fpi for fear of it burning a lot of my limited CPU, could you get instructions so I could set it up on nest when I have time?

Firepup650 commented

2024-04-20 06:52:16 +00:00

Author

Owner

As a side note, there's a reason there's low priority on this, I don't really want to bother setting it up, it'd just be nice to have.

Prevent branches from being merged that have a -dev suffix on the version number #2

Prevent branches from being merged that have a `-dev` suffix on the version number #2