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

Adapt player name extraction #463

Merged
merged 5 commits into from
Jun 10, 2021
Merged

Conversation

voelzmo
Copy link
Contributor

@voelzmo voelzmo commented Jun 4, 2021

This was changed with #283 in the previous chapter. Following through with this change in the following chapters to stay consistent.

@voelzmo
Copy link
Contributor Author

voelzmo commented Jun 5, 2021

Is a superset of #459

This was changed with quii#283 in the previous chapter. Following through with this change in the json chapter to stay consistent.
@voelzmo voelzmo force-pushed the adapt_player_name_extraction branch from 58e0eda to 08ccef4 Compare June 9, 2021 15:03
@voelzmo voelzmo changed the title Adapt player name extraction in json chapter Adapt player name extraction Jun 9, 2021
@quii
Copy link
Owner

quii commented Jun 10, 2021

Thanks @voelzmo

@quii quii merged commit d703543 into quii:main Jun 10, 2021
@voelzmo voelzmo deleted the adapt_player_name_extraction branch June 11, 2021 11:36
gypsydave5 pushed a commit to gypsydave5/learn-go-with-tests that referenced this pull request Jun 11, 2021
* Adapt player name extraction in json chapter

This was changed with quii#283 in the previous chapter. Following through with this change in the json chapter to stay consistent.

* Adapt player name extraction in io chapter

* Adapt player name extraction in cmdline chapter

* Adapt player name restriction in time chapter

* Adapt player name extraction in websocket chapter
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