Skip to content

Conversation

@jdetter
Copy link
Collaborator

@jdetter jdetter commented Jun 20, 2023

Description of Changes

  • Testsuite now runs clippy against the default module that is created with spacetime init

API

  • This is a breaking change to the module API
  • This is a breaking change to the ClientAPI

If the API is breaking, please state below what will break

Copy link
Contributor

@cloutiertyler cloutiertyler left a comment

Choose a reason for hiding this comment

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

Good idea! I like it.

@jdetter jdetter enabled auto-merge (squash) June 22, 2023 15:04
@jdetter jdetter merged commit 50e357a into master Jun 22, 2023
cloutiertyler pushed a commit that referenced this pull request Aug 1, 2023
* Clippy runs on default module

* Fix clippy issue with default module

---------

Co-authored-by: Boppy <[email protected]>
cloutiertyler pushed a commit that referenced this pull request Aug 1, 2023
* Clippy runs on default module

* Fix clippy issue with default module

---------

Co-authored-by: Boppy <[email protected]>
@cloutiertyler cloutiertyler deleted the jdetter/default-module-test branch August 1, 2023 21:54
bfops pushed a commit that referenced this pull request Jul 16, 2025
bfops pushed a commit that referenced this pull request Jul 17, 2025
* Updated server side module

* More simplification

* More progress

* More updates

* Ready to start testing again

* Got through tutorial with some issues, going again

* Added warnings

* Small fix, this is ready to be released

---------

Co-authored-by: John Detter <[email protected]>
bfops pushed a commit that referenced this pull request Jul 17, 2025
* Upload script to help deploying to our partners

* You have to specify hosts individually

* Small fix

* Another small fix - tested + working

---------

Co-authored-by: John Detter <[email protected]>
bfops pushed a commit that referenced this pull request Aug 7, 2025
bfops pushed a commit that referenced this pull request Aug 7, 2025
* Upload script to help deploying to our partners

* You have to specify hosts individually

* Small fix

* Another small fix - tested + working

---------

Co-authored-by: John Detter <[email protected]>
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.

3 participants