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

feat(koji): add koji #384

Merged
merged 2 commits into from
Jan 28, 2022
Merged

feat(koji): add koji #384

merged 2 commits into from
Jan 28, 2022

Conversation

its-danny
Copy link
Contributor

Not a clue what the requirements are for getting a package added, but would be nice to get my little conventional commit util in here if possible. No windows support atm, so no installer for it.

The project for reference: https://github.com/its-danny/koji

@coolaj86
Copy link
Member

Hey, just wanted to let you know that I appreciate this and I'll be taking a look at it within the next week or so.

I just spent a bunch of time on Webi PRs and updates a few weeks ago and I've needed to focus on some other projects for a bit to get caught up on them.

:)

Copy link
Member

@coolaj86 coolaj86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really awesome! It's up on beta at https://beta.webinstall.dev/koji

I bet @wmerfalen will love it (and we'll probably start using it at work).

That said, I have a few requests before merging this in. Mostly because webinstall.dev is meant for idiots like me and the docs aren't quite complete enough for me to know how I'm supposed to use this.

I'd love to get this into prod in the next day or two if you'd just make the couple of requested updates.

:)

koji/README.md Show resolved Hide resolved
koji/README.md Outdated Show resolved Hide resolved
koji/README.md Show resolved Hide resolved
koji/install.sh Outdated Show resolved Hide resolved
@coolaj86
Copy link
Member

P.S. Would you like to join us in remote for a lightning talk in February at Utah Rust? Someone there may be able to help with Windows support.

@its-danny
Copy link
Contributor Author

@coolaj86 heyo, will get to this tonight! i plan on adding windows support at some point, mostly comes down to not knowing how to have github actions build for windows yet but its on my todo list.

@its-danny
Copy link
Contributor Author

@coolaj86 I think all is well, fleshed out README and ran prettier & shfmt.

koji/README.md Outdated
cd dev/work-stuff
# Do some work
cd dev/work-stuff
git add .env.production
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

umm... no... DO NOT commit private keys for prod!!!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

joke from repo README lol, probably shoulda changed it

@coolaj86
Copy link
Member

Now that I understand this better and I've had a chance to try it out, I've updated the README in a way that I feel is more targeted towards our audience.

@coolaj86 coolaj86 merged commit 2acd20e into webinstall:main Jan 28, 2022
@coolaj86
Copy link
Member

Live at https://webinstall.dev/koji/

I don't have it on the homepage yet, but I think it needs a little work before it's ready for prime time. :)

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.

None yet

2 participants