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

ibus-engines.bamboo: 0.8.2-rc18 -> 0.8.4-rc3 #313316

Merged

Conversation

goatastronaut0212
Copy link
Member

@goatastronaut0212 goatastronaut0212 commented May 21, 2024

Description of changes

Newer version will fix issue in BambooEngine/ibus-bamboo#482

I also want to become a maintainer of this package.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` label May 21, 2024
@NixOSInfra NixOSInfra added the 12. first-time contribution This PR is the author's first one; please be gentle! label May 21, 2024
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels May 21, 2024
@nbraud
Copy link
Contributor

nbraud commented May 22, 2024

Thanks for the contribution, @goatastronaut0212 !

@nbraud
Copy link
Contributor

nbraud commented May 22, 2024

@SuperBo, I think your input would be welcome as well :)

@SuperBo
Copy link
Contributor

SuperBo commented May 22, 2024

@nbraud, I am very happy with @goatastronaut0212 work. I don't have access to a Linux machine anymore. So I think it would be better if we can remove my name out of maintainer list and let @goatastronaut0212 be the only maintainer.

@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label May 22, 2024
@wegank wegank removed the 12.approvals: 1 This PR was reviewed and approved by one reputable person label May 23, 2024
@goatastronaut0212
Copy link
Member Author

goatastronaut0212 commented May 23, 2024

Can you rereview my PR again? My commit seems a little late. @nbraud

I have removed @SuperBo maintainer in maintainer list from package. It is really sad to see he don't have much time and effort to continue with me, but I hope he will go well in life and thank you for his previous contribution. Your contribution really helps me a lot.

Copy link
Contributor

@Pandapip1 Pandapip1 left a comment

Choose a reason for hiding this comment

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

Diff LGTM

@Pandapip1 Pandapip1 added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label May 23, 2024
Copy link
Contributor

@nbraud nbraud left a comment

Choose a reason for hiding this comment

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

I'll squash some of your commits together while merging

@nbraud nbraud force-pushed the update-ibus-bamboo-and-add-maintainer branch from 40c17fd to 6536fbb Compare May 26, 2024 17:51
@nbraud
Copy link
Contributor

nbraud commented May 26, 2024

Result of nixpkgs-review pr 313316 run on x86_64-linux 1

1 package built:
  • ibus-engines.bamboo

@nbraud nbraud merged commit 124fa58 into NixOS:master May 26, 2024
7 of 8 checks passed
@nbraud nbraud added 12.approvals: 2 This PR was reviewed and approved by two reputable people and removed 12.approvals: 1 This PR was reviewed and approved by one reputable person labels May 26, 2024
@goatastronaut0212 goatastronaut0212 deleted the update-ibus-bamboo-and-add-maintainer branch May 27, 2024 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes 12.approvals: 2 This PR was reviewed and approved by two reputable people 12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants