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

creep.say functions #209

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

couchcactus
Copy link

added creep,say functions (not very good for low CPU)

added creep,say functions (not very good for low CPU)
@couchcactus
Copy link
Author

not sure if it will be merged, there is promises in the code of low CPU optimisation, creep.say isn't very good for that

@couchcactus
Copy link
Author

withdraw, transport, build, upgrade, attack, dismantle, power functions and etc now have creep.say functions

@JonathanSafer JonathanSafer self-requested a review May 18, 2024 03:54
Copy link
Owner

@JonathanSafer JonathanSafer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is a reasonable idea, but you need to make the change to the source of this repository. Adding this 11k line file won't change anything on the source.

Steps to modify the source:

  1. fork the repo (on this page at the top right above the green code button
  2. clone your fork git clone https://github.com/couchcactus/screeps.git
  3. open the code in an editor (vscode, sublime text, notepad++...)
  4. make your changes (only change files that end with .ts, everything else is autogenerated)
  5. test your changes with npm test (see README.md for more info)
  6. commit & push your changes
  7. open github.com and open a pull request

If this sounds like a lot of work just to suggest a change, it is. But this will save you a lot of time in the future and help keep things organized.

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