airbnb / babel-plugin-inline-react-svg

A babel plugin that optimizes and inlines SVGs for your React Components.
MIT License
474 stars 92 forks source link

migrate from travis to github actions #117

Closed akellbl4 closed 1 year ago

akellbl4 commented 2 years ago

Implements CI jobs with Github Actions Passed actions on my fork

ljharb commented 2 years ago

@akellbl4 the only other thing this needs is the "restricted branches" setting changed to no longer require travis, which requires an @airbnb employee to change.

Once this lands, the actions will run, after which the restricted branch settings can add the actions it should thenceforth require.

ljharb commented 2 years ago

Rebased; this is now good to go. No need for further changes until an admin is able to fix the protected branch settings, at which point i'll merge this :-)

akellbl4 commented 1 year ago

Hey @ljharb, how can we ping anyone from @airbnb to take a look on it? :)

ljharb commented 1 year ago

I’ve already done so; all we can do now is wait.

lencioni commented 1 year ago

@ljharb I've added some of the new checks as required in the branch restriction settings, but I seem to be missing a node one that we've done elsewhere.

image

Do you happen to know what I should be searching for here? Is it posttest?

ljharb commented 1 year ago

@lencioni there should be posttest, as well as two node ones, but they won't show up in the autocomplete unless they've run recently, and this PR ran months ago :-) the merge into master, however, will populate the dropdown soon so you can add them.

lencioni commented 1 year ago

The node ones are not showing up. I had done #119 in hopes that it would trigger it, but it seems to not have. In any case, I added posttest now.

ljharb commented 1 year ago

ah, try "latest majors"

lencioni commented 1 year ago

That gives me a check for each version, unfortunately

image
ljharb commented 1 year ago

hm, you're right. i'll make a PR to fix it.

ljharb commented 1 year ago

120

lencioni commented 1 year ago

Thanks! I added that one to the settings now, so I think we can call this a wrap

ljharb commented 1 year ago

@lencioni one more thing :-) can we remove the review requirement? i need to be able to push things without a PR to release properly.

lencioni commented 1 year ago

@ljharb agreed, that is super annoying for releasing. I think I've updated the settings now.

ljharb commented 1 year ago

Thanks! The only thing left is that the automatic rebase and allow edits checks only run on a PR :-/ so either we can make those optional, or, add me to the airbnb org with a custom role that allows me to bypass branch protections (which i assume is a bit more difficult, bureaucratically)

lencioni commented 1 year ago

I have made those optional now too. I think that's fine since making them required is unlikely to matter all that much anyway.

ljharb commented 1 year ago

Much obliged, and I agree.