Skip to content
This repository was archived by the owner on Apr 14, 2023. It is now read-only.

README: Fix more typos#69

Closed
DeathlyPlays wants to merge 3 commits into
AgustinSRG:masterfrom
DeathlyPlays:patch-2
Closed

README: Fix more typos#69
DeathlyPlays wants to merge 3 commits into
AgustinSRG:masterfrom
DeathlyPlays:patch-2

Conversation

@DeathlyPlays

Copy link
Copy Markdown

No description provided.

@Mystifi

Mystifi commented Jun 27, 2017

Copy link
Copy Markdown
Contributor

Several of these typos are repeated throughout repository code, especially the typo with "inappropriate." It might be worth checking for them in the source?

@DeathlyPlays

Copy link
Copy Markdown
Author

Perhaps @Mystifi afaik Ecuacion isn't resuming coding this so I might try continuing on a new repo adding more to this

@TheRun98

TheRun98 commented Jun 29, 2017

Copy link
Copy Markdown
Contributor

Please do, there have been issues from Object.merge lately. If its done I can definitely try to help contribute. The last real major update was with battle mechanics and simple typos I resolved when starting up the bot.

@DeathlyPlays

Copy link
Copy Markdown
Author

@TheRun98 I used the recent pr's made that work and added them to my github for this as well like #68

@DeathlyPlays

Copy link
Copy Markdown
Author

Meh I'll add #67 as well

@Mystifi

Mystifi commented Jun 29, 2017

Copy link
Copy Markdown
Contributor

Please do, there have been issues from Object.merge lately.

@TheRun98, you should be able to use Object.assign everywhere Object.merge is used because the latter is simply sugar.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants