Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removing old config ini file based scripts #226

Merged
merged 2 commits into from
Jun 15, 2024

Conversation

shimwell
Copy link
Collaborator

Description

we no longer need this folder due to the geouned_cadtocsg script being installed along with the package and the removal of config ini file support

Checklist

  • I'm making a PR from a feature branch on my fork into GEOUNED-org/GEOUNED/dev branch
  • I have followed PEP8 style guide for Python or run a formatter such as black or ruff format on my code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@shimwell shimwell changed the base branch from main to dev June 14, 2024 08:17
@shimwell shimwell marked this pull request as draft June 14, 2024 08:22
@shimwell
Copy link
Collaborator Author

waiting to merge this PR till we get reverse (CsgToCad) working and tested

@shimwell
Copy link
Collaborator Author

Now that #229 is ready we can consider this PR

@shimwell shimwell marked this pull request as ready for review June 14, 2024 19:19
@psauvan psauvan merged commit b2ab4fe into GEOUNED-org:dev Jun 15, 2024
9 checks passed
psauvan pushed a commit that referenced this pull request Jun 17, 2024
* removed old scripts

* retrigger ci
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants