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

make-package.sh: don't attempt to include matchmaker #943

Merged
merged 2 commits into from
Dec 21, 2022

Conversation

juped
Copy link
Member

@juped juped commented Dec 21, 2022

The matchmaker has been removed, but the packaging script still attempts to include it. This does not break the script, because the name of the nonexistent matchmaker is the empty string, but is still incorrect.

The matchmaker has been removed, but the packaging script still
attempts to include it. This does not break the script, because the
name of the nonexistent matchmaker is the empty string, but is still
incorrect.
@juped juped requested a review from Fraccaman December 21, 2022 06:29
@juped juped marked this pull request as ready for review December 21, 2022 06:29
@juped juped merged commit 077f088 into main Dec 21, 2022
@juped juped deleted the ray/no-matchmaker-in-package branch December 21, 2022 12:57
juped added a commit that referenced this pull request Dec 22, 2022
This cherry-picks the merges of branches
'namada/ray/no-matchmaker-in-package' (#943) and
'namada/ray/release-include-license' (#945) into main, as merged in
commit 077f088.

I was an incompetent maintainer here, and if you are reading this, you
are paying the price.

* namada/ray/no-matchmaker-in-package:
  changelog: add #943
  make-package.sh: don't attempt to include matchmaker

* namada/ray/release-include-license:
  changelog: add #945
  ci: install cargo-about on release step
  make-package.sh: include license information in tarball
  release: add configuration for cargo-about
tzemanovic added a commit that referenced this pull request Dec 28, 2022
* origin/grarco/basic-fee:
  changelog: add #962
  Fixes unit tests
  Fixes process proposal fee token
  Fixes fee value in tx contruction
  Adds fee burning and checks
  changed to 0.12.2
  Namada 0.12.2
  release: cherry-pick release improvements
  changelog: add #952
  prepare_proposal: update comment to reflect hotfix
  prepare_proposal: use TxRecord struct on ABCI++ build
  missed a chain-id addition
  Prioritize wrapper txs during proposal construction
  Hot fix PrepareProposal
  added chain id and fixed install docs
  update pseudocode
  General housekeeping
  Reward distribution: general improvements
  Cubic slashing: general writing improvements
  Cubic slashing: add new improved pseudocode blurb (Rust-style)
  Cubic slashing: clarify slash iteration procedure and equation
  Cubic slashing: fix and improve plot
  quick fix to cubic slash
  changelog: add #911
  changelog: add #876
  test/core: run less cases on `update_epoch_after_its_duration` test
  changelog: add #945
  ci: install cargo-about on release step
  make-package.sh: include license information in tarball
  release: add configuration for cargo-about
  changelog: add #943
  make-package.sh: don't attempt to include matchmaker
  Namada 0.12.1
  changelog: add #942
  vp_verify_masp: avoid panicking unwrap()s
  changed the docker link (again)
  Changed the first header of pages to H1 in 2 occasions in Namada Specs
  Replaced non-breaking space with space to fix faulty formatting
  fix transfer args
bengtlofgren pushed a commit that referenced this pull request May 11, 2023
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.

2 participants