Draft: Godot4update

[tlobig]

Please check if the PR fulfills these requirements:

• [ ] The commit message follows our guidelines.
• For bug fixes and features:
  • [ ] You tested the changes.

Related issue (if applicable): #10

What kind of change does this PR introduce? Course demo code runs in Godot 4.0 stable

Does this PR introduce a breaking change? Breaks compatibility with Godot 3.x

New feature or change

All code should work with Godot 4.x (4.0 stable as of writing)

TODO

• BasicDungeon
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• MSTDungeon
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• RandomWalker
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• WorldMap
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• ModularWeapons
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• SpaceInfiniteGeneration
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• CellularAutomata
  • [x] ported
  • [ ] reviewed
  • [ ] cleaned
• [ ] start project templates ported
• [ ] commits squashed

[tlobig]

PR is still in draft, last two parts of the course are missing

commits should be squashed and comments cleaned before merging

[NathanLovato]

Thanks much! Just a heads up, I'll be away for the weekend, only working on project management and email/communication. I'll do code reviews again after that.

[tlobig]

I included this suggested bugfix in the SpaceInfiniteGeneration/PersistentWorldGenerator part, after amending it to work properly: https://gdquest.mavenseed.com/community/14469-bug-in-_find_sector

[tlobig]

first pass done, review of changes can begin.

[NathanLovato]

Just asked Razvan, he confirmed he'll review this as I currently have too much on my plate (still crunching, not very efficient anymore). He works from Thursday to Saturday so please note it might be a couple of days before he gets to this.

[NathanLovato]

Merged as #12