Skip to content

README update - #164

Open
QCplus wants to merge 1 commit into
chickensoft-games:mainfrom
QCplus:main
Open

README update#164
QCplus wants to merge 1 commit into
chickensoft-games:mainfrom
QCplus:main

Conversation

@QCplus

@QCplus QCplus commented Jul 12, 2025

Copy link
Copy Markdown

As I understand correctly there are 3 steps to make GoDotTest work:

  1. Install nuget package
  2. Create a node that invokes tests if appropriate parameters were given else run game
  3. Add VS code config to run debug

It was confusing for me at first that step 2 is after 3 in README. What do you think about moving test node creation info to the 'Setup' section?

Also link to the 'TestEnvironment' class and GoDotTest namespaces were old

@wlsnmrk wlsnmrk added the documentation Improvements or additions to documentation label Jul 24, 2025
@wlsnmrk
wlsnmrk requested a review from jolexxa July 24, 2025 15:11

@wlsnmrk wlsnmrk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the link/namespace fixes! I can see the argument for re-ordering the sections as you suggest, but I'm going to ask @jolexxa to take a quick look in case she has any thoughts on that.

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants