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.

Sign up for GitHub

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

Jump to bottom

Tools: reference dist in team-gen-benchmark script #10777

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

Tools: reference dist in team-gen-benchmark script #10777

wants to merge 1 commit into from

Conversation

DieterReinert
Copy link
Contributor

@DieterReinert DieterReinert commented Dec 31, 2024 •
edited
Loading

This PR addresses #10644.

Developers just need to run node build to rebuild the dist folder whenever the .ts files in sim/ change.

Other Considerations

  • Add a watch script to rebuild when .ts files change.
    • npx tsc --watch
  • Convert the script itself to TypeScript & compile it with tsc.
  • Convert the script itself to TypeScript & use ts-node + nodemon.

@DieterReinert
tools: reference dist/ folder in team-generation-benchmark script (c…
3b039dc 
…loses smogon#10644)

This PR addresses [smogon#10644](smogon#10644).

Developers just need to run `node build` to rebuild the `dist` folder whenever the `.ts` files in `sim/` change.

## Other Considerations
- Add a watch script to rebuild when ``.ts`` changes.
- Convert the script itself to TypeScript & compile it with ``tsc``.
- Convert the script itself to TypeScript & use ``ts-node``  + ``nodemon``.
@DieterReinert DieterReinert changed the title tools: reference dist/ folder in team-generation-benchmark script (… tools: reference dist/ folder in team-generation-benchmark script Dec 31, 2024
@DieterReinert DieterReinert changed the title tools: reference dist/ folder in team-generation-benchmark script Tools: reference dist in team-gen-benchmark script Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DieterReinert