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.

Sign up for GitHub

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

Jump to bottom

225 irods folder #226

Merged
merged 5 commits into from
Jul 10, 2024
Merged

225 irods folder #226

merged 5 commits into from
Jul 10, 2024

Conversation

chStaiger
Copy link
Collaborator

Two small things:

  • create .irods folder for user, if it does not exist
  • Add a line in the IrodsPath tutorials showing how to create a collection

@chStaiger chStaiger requested a review from qubixes July 4, 2024 16:13
@chStaiger chStaiger linked an issue Jul 4, 2024 that may be closed by this pull request
Create .irods folder in CLI #225
Closed
qubixes
qubixes requested changes Jul 5, 2024
View reviewed changes
Copy link
Collaborator

@qubixes qubixes left a comment

Choose a reason for hiding this comment

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

If we're just using it in one place, and it's basically a one-liner. I suggest we just put the one-liner in the main function:

(Path.home() / ".irods").mkdir(exist_ok=True)

qubixes
qubixes reviewed Jul 8, 2024
View reviewed changes
ibridges/__main__.py Outdated Show resolved Hide resolved
@chStaiger @qubixes
Update ibridges/__main__.py
dbefd0f 
Co-authored-by: qubixes <44498096+qubixes@users.noreply.github.com>
@qubixes qubixes merged commit fdfceaf into develop Jul 10, 2024
9 checks passed
@qubixes qubixes deleted the 225-irods-folder branch July 10, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qubixes qubixes qubixes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create .irods folder in CLI
2 participants
@chStaiger @qubixes