Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Add Windows support for storage monitor #13466

Merged

Conversation

ark0f
Copy link
Contributor

@ark0f ark0f commented Feb 24, 2023

No description provided.

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Feb 24, 2023

User @ark0f, please sign the CLA here.

@bkchr bkchr added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit A0-pleasereview labels Feb 25, 2023
client/storage-monitor/src/lib.rs Outdated Show resolved Hide resolved
client/storage-monitor/src/lib.rs Outdated Show resolved Hide resolved
@nazar-pc
Copy link
Contributor

I'd like to request faster upstreaming here, without this sc-service is not possible to compile on Windows either.

P.S. Would be great to have basic cargo c on Windows platform in CI.

@bkchr
Copy link
Member

bkchr commented Feb 27, 2023

I'd like to request faster upstreaming here, without this sc-service is not possible to compile on Windows either.

P.S. Would be great to have basic cargo c on Windows platform in CI.

We are waiting for @ark0f to accept the proposed changes and then we can merge it. We don't have permissions to commit to his branch.

@ark0f
Copy link
Contributor Author

ark0f commented Feb 27, 2023

Oops. I'm sorry. Rebased on wrong master branch

@ggwpez
Copy link
Member

ggwpez commented Feb 27, 2023

Please dont force push. You can just do git merge origin/master and then push that, see Contributing guide.

@ggwpez
Copy link
Member

ggwpez commented Feb 27, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 3d40b17 into paritytech:master Feb 27, 2023
ukint-vs pushed a commit to gear-tech/substrate that referenced this pull request Apr 10, 2023
* Add Windows support for storage monitor

* Apply suggested changes
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* Add Windows support for storage monitor

* Apply suggested changes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants