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

Add start node command to client #1728

Closed
wants to merge 2 commits into from

Conversation

massa-bot
Copy link

In GitLab by @yvan-sraka

Closes #159

@yvan-sraka yvan-sraka changed the base branch from dev to master November 14, 2021 22:30
@yvan-sraka yvan-sraka self-assigned this Nov 15, 2021
@AureliaDolo
Copy link
Contributor

@yvan-sraka this issue with the config files wasn't solved ?

@yvan-sraka yvan-sraka changed the title Draft: Resolve "Add start node command to client" Add start node command to client Nov 15, 2021
@yvan-sraka
Copy link
Contributor

Not yet :/

@AureliaDolo
Copy link
Contributor

@yvan-sraka so this issue is blocked until the config related ones aren't solved ?

@yvan-sraka
Copy link
Contributor

Yes! Blocked by #1277

@yvan-sraka yvan-sraka added the blocked Issues that can't be done for now. label Nov 18, 2021
@damip
Copy link
Member

damip commented Nov 23, 2021

This is actually hard to make in a clean and portable way... maybe let's close it for now

@yvan-sraka
Copy link
Contributor

It's easy to do it in a clean and portable way! I will come with a solution this week :)

@AureliaDolo
Copy link
Contributor

@yvan-sraka is there a bot that is able to ping us in a week ?

@yvan-sraka
Copy link
Contributor

Just need #1848 to be merged and we're good :)

@AureliaDolo
Copy link
Contributor

This issue is still blocked right ?

@AureliaDolo AureliaDolo linked an issue Dec 7, 2021 that may be closed by this pull request
@AureliaDolo AureliaDolo added todo and removed blocked Issues that can't be done for now. labels Dec 7, 2021
@yvan-sraka
Copy link
Contributor

This issue is still blocked, right?

Not anymore! Since #1848 have been merged :)

@yvan-sraka yvan-sraka removed the dying label Mar 9, 2022
@yvan-sraka yvan-sraka marked this pull request as draft March 9, 2022 09:58
@yvan-sraka yvan-sraka marked this pull request as ready for review March 25, 2022 15:03
@yvan-sraka yvan-sraka force-pushed the 159-add-start-node-command-to-client branch from 7728752 to 9647b7c Compare April 25, 2022 10:02
@yvan-sraka yvan-sraka requested review from damip and AurelienFT April 25, 2022 10:03
@yvan-sraka yvan-sraka force-pushed the 159-add-start-node-command-to-client branch from 9647b7c to 6193598 Compare April 25, 2022 10:05
@yvan-sraka yvan-sraka force-pushed the 159-add-start-node-command-to-client branch from 6193598 to 25bdaaa Compare April 25, 2022 10:06
@AurelienFT
Copy link
Contributor

Can you fix the CI before review process please ? @yvan-sraka

@AurelienFT AurelienFT closed this May 3, 2022
@AurelienFT AurelienFT deleted the 159-add-start-node-command-to-client branch May 29, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Add start node command to client Implement ban nodes workflow between consensus and protocol.
5 participants