dcronqvist / DotTiled

📚 A .NET library for loading, saving, and handling Tiled maps and tilesets
https://dcronqvist.github.io/DotTiled
MIT License
12 stars 2 forks source link

Example project and updated .gitignore #32

Closed krnlexception closed 1 month ago

krnlexception commented 1 month ago

Description

Type of Change

Checklist

Simple example project with a few common usages, eg. reading tiles, properties, tilesets. Example includes both quick and easy way to load, and manually. Also updated .gitignore to ignore .idea directory created by JetBrains Rider.

krnlexception commented 1 month ago

Changes made, force pushed before because I haven't changed username and e-mail in git (forgot to do that after changing my username). Used embedded resources in manual example.

make lint returns now

/home/krnl/Projects/DotTiled/src/DotTiled.Example/Program.cs(6,7): warning IDE0040: Accessibility modifiers required [/home/krnl/Projects/DotTiled/src/DotTiled.Example/DotTiled.Example.csproj]
/home/krnl/Projects/DotTiled/src/DotTiled.Example/Program.cs(8,3): warning IDE0210: Convert to top-level statements [/home/krnl/Projects/DotTiled/src/DotTiled.Example/DotTiled.Example.csproj]

which I can't really fix, first one is class definition, second one is because I don't use top-level statements.

I can also make examples for other features, or using actual game engines, eg. Unity, or Godot.

krnlexception commented 1 month ago

Godot 4.3 supports full .NET 8 capabilities, so unless there's a bug, DotTiled will work. I'll try later, and if it works, I'll put Godot example with this PR

dcronqvist commented 1 month ago

Godot 4.3 supports full .NET 8 capabilities, so unless there's a bug, DotTiled will work. I'll try later, and if it works, I'll put Godot example with this PR

Interesting! I was not aware of that, let me know if it works out :)

krnlexception commented 1 month ago

Done, working on Godot example now. Once I get it working, I'll push it to this PR.

krnlexception commented 1 month ago

Godot example done, PR can be merged.