Skip to content

Conversation

@csfore
Copy link
Contributor

@csfore csfore commented Sep 22, 2023

Made sure to run test.sh before submitting and all tests pass.

@xxc3nsoredxx
Copy link
Contributor

Looks ok otherwise. Personally, I'd also squash the commits.

@csfore
Copy link
Contributor Author

csfore commented Oct 14, 2023

Cleaned it up! How's it look?

@xxc3nsoredxx
Copy link
Contributor

You also clobbered the commit message when squashing. If you could make it something useful, that'd be nice.

@csfore
Copy link
Contributor Author

csfore commented Oct 15, 2023

Would "Add new service - paste.gentoo.zip" suffice?

@xxc3nsoredxx
Copy link
Contributor

Sure. Maybe mention in the message body that you specify pgz on the cli to use it.

Specified in the code with `pgz` and in the cli for usage and brevity.
@csfore
Copy link
Contributor Author

csfore commented Oct 15, 2023

How's that look?

@xxc3nsoredxx
Copy link
Contributor

LGTM

xxc3nsoredxx added a commit to xxc3nsoredxx/gentoo that referenced this pull request Oct 15, 2023
From zlin/wgetpaste#48

Signed-off-by: Oskari Pirhonen <xxc3ncoredxx@gmail.com>
@heirecka heirecka merged commit f1906f0 into zlin:master Oct 18, 2023
@heirecka
Copy link
Collaborator

Thanks!

gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Oct 19, 2023
From zlin/wgetpaste#48

Signed-off-by: Oskari Pirhonen <xxc3ncoredxx@gmail.com>
Closes: #33340
Signed-off-by: Florian Schmaus <flow@gentoo.org>
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.

3 participants