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

sc-tracing: enable env-filter feature #4357

Merged
merged 2 commits into from
May 2, 2024
Merged

Conversation

Morganamilo
Copy link
Contributor

This crate uses this feature however it appears to still work without this feature enabled. I believe this is due to feature unification of the workspace. Some other crate enables this feature so it also ends up enabled here. But when this crate is pushed to crates.io and compiled individualy it fails to compile.

This crate uses this feature however it appears to still work without
this feature enabled. I believe this is due to feature unification of
the workspace. Some other crate enables this feature so it also ends up
enabled here. But when this crate is pushed to crates.io and compiled
individualy it fails to compile.
@Morganamilo Morganamilo requested a review from koute as a code owner May 2, 2024 12:24
@Morganamilo Morganamilo requested a review from a team May 2, 2024 12:24
@Morganamilo Morganamilo added the A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). label May 2, 2024
@skunert skunert requested a review from a team May 2, 2024 12:33
@Morganamilo Morganamilo added the R0-silent Changes should not be mentioned in any release notes label May 2, 2024
@Morganamilo Morganamilo enabled auto-merge May 2, 2024 13:03
@Morganamilo Morganamilo added this pull request to the merge queue May 2, 2024
Merged via the queue into master with commit df84ea7 May 2, 2024
134 of 136 checks passed
@Morganamilo Morganamilo deleted the morganamilo/env-filter branch May 2, 2024 14:01
dcolley added a commit to metaspan/polkadot-sdk that referenced this pull request May 6, 2024
* 'master' of https://github.com/metaspan/polkadot-sdk: (65 commits)
  Introduces `TypeWithDefault<T, D: Get<T>>` (paritytech#4034)
  Publish `polkadot-sdk-frame`  crate (paritytech#4370)
  Add validate field to prdoc (paritytech#4368)
  State trie migration on asset-hub westend and collectives westend (paritytech#4185)
  Fix: dust unbonded for zero existential deposit (paritytech#4364)
  Bridge: added subcommand to relay single parachain header (paritytech#4365)
  Bridge: fix zombienet tests (paritytech#4367)
  [WIP][CI] Add more GHA jobs (paritytech#4270)
  Allow for 0 existential deposit in benchmarks for `pallet_staking`, `pallet_session`, and `pallet_balances` (paritytech#4346)
  Deprecate `NativeElseWasmExecutor` (paritytech#4329)
  More `xcm::v4` cleanup and `xcm_fee_payment_runtime_api::XcmPaymentApi` nits (paritytech#4355)
  sc-tracing: enable env-filter feature (paritytech#4357)
  deps: update jsonrpsee to v0.22.5 (paritytech#4330)
  Add PoV-reclaim enablement guide to polkadot-sdk-docs (paritytech#4244)
  cargo: Update experimental litep2p to latest version (paritytech#4344)
  Bridge: ignore client errors when calling recently added `*_free_headers_interval` methods (paritytech#4350)
  Make parachain template great again (and async backing ready) (paritytech#4295)
  [Backport] Version bumps and reorg prdocs from 1.11.0 (paritytech#4336)
  HRMP - set `DefaultChannelSizeAndCapacityWithSystem` with dynamic values according to the `ActiveConfig` (paritytech#4332)
  Statement Distribution Per Peer Rate Limit (paritytech#3444)
  ...
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
This crate uses this feature however it appears to still work without
this feature enabled. I believe this is due to feature unification of
the workspace. Some other crate enables this feature so it also ends up
enabled here. But when this crate is pushed to crates.io and compiled
individualy it fails to compile.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants