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

Python script for Changelog auto-completion #531

Merged
merged 3 commits into from
Jun 6, 2019
Merged

Conversation

Jaskar
Copy link
Contributor

@Jaskar Jaskar commented Jun 4, 2019

Python script for Changelog auto-completion, retrieving datas from the Github.

  • Changelog : Last version date added (needed for auto completion)
  • Readme : Explaination added
  • BabylonExporter.cs : Version number incremented by 0.1 for testing purpose
  • Merge from official repo in mine

Jaskar added 2 commits June 4, 2019 17:59
…e Github.

- Changelog : Last version date added (needed for auto completion)
- Readme : Explaination added
- BabylonExporter.cs : Version number incremented by 0.1 for testing purpose
@deltakosh deltakosh requested a review from Drigax June 4, 2019 16:42
Copy link
Contributor

@Drigax Drigax left a comment

Choose a reason for hiding this comment

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

Where do we expect our users to execute this script from?

I see that we're using pathing from the 3dsMax solution folder root to the Exporter/BabylonExporter file to detect our new version number, but I originally wrote this to path from the Max2Babylon project folder root containing the script. Not a huge problem, but just want to make sure that this works as expected and is documented as such.

3ds Max/Max2Babylon/Max2Babylon_Package.py Outdated Show resolved Hide resolved
3ds Max/Max2Babylon/Max2Babylon_Package.py Show resolved Hide resolved
@Jaskar
Copy link
Contributor Author

Jaskar commented Jun 6, 2019

This should be good !

@Drigax
Copy link
Contributor

Drigax commented Jun 6, 2019

Thanks, looks great! Checking in.

@Drigax Drigax merged commit bbabd59 into BabylonJS:master Jun 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants