Skip to content

Bringing the name of the example projects to a single name #154

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

Merged
merged 3 commits into from
May 19, 2025

Conversation

aglitchman
Copy link
Contributor

In the root of the project I added a common settings file for all examples, which is applied when building projects via bob.jar. For this purpose, it is copied to the example directory when building it (note: if we specify an absolute path to the settings file, bob.jar does not want to use it because it is outside the example directory).

Closes #153

@aglitchman
Copy link
Contributor Author

If PR is merged, please trigger rebuild of the examples to update https://defold.com/examples/material/custom_sprite/ with proper files.

@britzl
Copy link
Collaborator

britzl commented May 16, 2025

Do we anticipate more common settings than the project title? If not, then a simple replace_in_file() should suffice:

game_project = os.path.join(example_src_dir, "game.project")
replace_in_file(game_project, r"title = .*", r"title = Defold-example")

@aglitchman
Copy link
Contributor Author

Do we anticipate more common settings than the project title? If not, then a simple replace_in_file() should suffice:

I was thought I could add something else to the settings file (in the future). For now it is not needed, then let it be just an title replacement in the game.project file 🙌

Also, please, then merge the PR - defold/examples#78

@aglitchman aglitchman changed the title Add settings file for examples Bringing the name of the example projects to a single name May 19, 2025
Copy link
Collaborator

@britzl britzl left a comment

Choose a reason for hiding this comment

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

TY

@britzl britzl merged commit b06c3f0 into defold:master May 19, 2025
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.

Allow to use any project name for the example
2 participants