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.

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

Documentation: document map parsing design #7633

Merged
merged 1 commit into from
Sep 14, 2020

Conversation

DanVanAtta
Copy link
Member

Documents the discussion started in: #7498,
covers the terms parsing and game data assembly and describes the design
philosophy that map-reading should just read the contents of XML and assembly
converts the XML model objects into actual java game objects.

Testing

Screens Shots

Additional Notes to Reviewer

Release Note

Documents the discussion started in: #7498,
covers the terms parsing and game data assembly and describes the design
philosophy that map-reading should just read the contents of XML and assembly
converts the XML model objects into actual java game objects.
@codecov
Copy link

codecov bot commented Sep 12, 2020

Codecov Report

Merging #7633 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #7633   +/-   ##
=========================================
  Coverage     24.32%   24.32%           
  Complexity     7084     7084           
=========================================
  Files          1211     1211           
  Lines         79025    79025           
  Branches      11319    11319           
=========================================
  Hits          19221    19221           
  Misses        57634    57634           
  Partials       2170     2170           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca7a4b4...708f6c5. Read the comment docs.

@Cernelius
Copy link
Contributor

Since even a developer misunderstood what I wrote down, chances are that I'm not being very clear. Unfortunately, I don't know how to present it better, but I'm open to suggestions (just someone tell me how it would reword it and I can at least check the substance is not adulterated under my understanding).

If it creates more confusion than help, the "Note" section can be completely removed (it just reproposes items, thus it is unnecessary). Just I made it because I assume otherwise people will assume that if their maps work after updating that means they updated it correctly (which, sadly, it's not the case, as the program doesn't fully validate everything).

@DanVanAtta
Copy link
Member Author

@Cernelius wrong PR on your last comment : )

See my last few comments in: #7630, I think the trick is to convert the document into a legend of deprecated/old items and what their latest should be

@RoiEXLab RoiEXLab merged commit 1c60d87 into master Sep 14, 2020
@RoiEXLab RoiEXLab deleted the document-map-parsing-design branch September 14, 2020 15:32
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