Skip to content

Conversation

@dansarie
Copy link
Contributor

@dansarie dansarie commented Nov 5, 2024

This adds roughtime.se to the ecosystem.

@cjpatton cjpatton self-requested a review December 2, 2024 15:35
Copy link
Contributor

@cjpatton cjpatton 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! Merge is currently blocked by the linter, though it seems unrelated to your change. Feel free to put up a PR to fix in the meantime. Otherwise, I'll look into this in the next couple of weeks.

@armfazh
Copy link

armfazh commented Dec 2, 2024

@dansarie you may need to rebase your PR on top of the updated master branch.

@cjpatton
Copy link
Contributor

cjpatton commented Dec 3, 2024

@dansarie you may need to rebase your PR on top of the updated master branch.

Ah yes, it looks like the branch is missing 260ff2b, which ought to fix this.

@dansarie dansarie closed this by deleting the head repository Dec 3, 2024
@dansarie
Copy link
Contributor Author

dansarie commented Dec 3, 2024

Accidentally deleted the head repository for this PR. New attempt in PR #66. Sorry about this!

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