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

Allow function definitions with slashes #137

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MysteryBlokHed
Copy link
Owner

@MysteryBlokHed MysteryBlokHed commented Sep 4, 2021

Closes #136

Allows functions to be defined with slashes in the name to create functions in subdirectories. This avoids being forced to split your code further if you want to have a function with a prefix like cmd/.

  • Make it work
  • Add tests
    • Test placement of files
    • Test contents of tags

Closes #136

Allows functions to be defined with slashes in the name to create
functions in subdirectories. This avoids being forced to split your code
further if you want to have a function with a prefix like `cmd/`.
@MysteryBlokHed MysteryBlokHed added this to the Databind 0.8 milestone Sep 4, 2021
@github-actions github-actions bot added cli Related to the Databind CLI source Affects the project's source labels Sep 4, 2021
Updates the function check_files_exist to support passing a files list
with folders in the paths.
Adds a test to cli_file_structure_tests.rs that makes sure that
functions with one or more subdirectories in the name are generated in
the right places.
@github-actions github-actions bot added the tests Affects the project's tests label Sep 5, 2021
@MysteryBlokHed
Copy link
Owner Author

The name chosen for the test directory means that #140 must be closed before this PR can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the Databind CLI source Affects the project's source tests Affects the project's tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow function definitions with slashes
1 participant