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

Moved the mock_imports file into the dea-notebooks repo #1229

Merged
merged 4 commits into from
May 23, 2024

Conversation

benji-glitsos-ga
Copy link
Collaborator

The mock_imports.txt file was previously in the dea-knowledge-hub repo, but that was inconvenient since actually it relates to the DEA Tools pages, so I moved it into this repo so that it will be easier to edit in future.

@benji-glitsos-ga benji-glitsos-ga changed the title Feature/moving mock imports file Moved the mock_imports file into the dea-notebooks repo May 22, 2024
Copy link
Collaborator

@BexDunn BexDunn left a comment

Choose a reason for hiding this comment

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

Looks good - worth mentioning somewhere in the readme for dea-notebooks what this one is for too?

@benji-glitsos-ga
Copy link
Collaborator Author

I have put on my todo list to write a Wiki article for how to create/update these DEA Tools Python modules since we don't currently have a Wiki article on this.

@benji-glitsos-ga benji-glitsos-ga merged commit c6116e6 into develop May 23, 2024
1 check passed
@benji-glitsos-ga benji-glitsos-ga deleted the feature/moving-mock-imports-file branch May 23, 2024 02:16
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