Skip to content

Conversation

@jxcl
Copy link
Contributor

@jxcl jxcl commented Dec 10, 2015

No description provided.

@rockymeza
Copy link

Oh yes definitely, this is what I needed before I could even start to use ogmios.

@acatton acatton added this to the 1.0 milestone Dec 14, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure to understand what you're trying to do here? os.path.join(x) is a no-op as far as I know (I could be wrong). Did you mean os.path.abspath(x)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's actually from you. I thought so as well but figured that you knew something I didn't. See 2ea20c8.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yep. That's a mistake from my side. Can you do another commit "fix antoine's shit" where you change this to os.path.abspath()? Thanks.

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.

4 participants