Skip to content

Conversation

@dimecon
Copy link
Contributor

@dimecon dimecon commented Jul 29, 2025

No description provided.

@dimecon dimecon requested a review from dc-mak July 29, 2025 14:04
@dc-mak
Copy link
Contributor

dc-mak commented Jul 29, 2025

I mean if it works it works but why not just point/link to the CI script?

@dc-mak
Copy link
Contributor

dc-mak commented Jul 29, 2025

Also using opam mean that local uncommitted changes are not picked up, which may not be what the user wants when trying to run locally?

@dimecon
Copy link
Contributor Author

dimecon commented Jul 29, 2025

I mean if it works it works but why not just point/link to the CI script?

Maybe that would better, but at least this extracts the key lines from it.

@dimecon
Copy link
Contributor Author

dimecon commented Jul 29, 2025

Also using opam mean that local uncommitted changes are not picked up, which may not be what the user wants when trying to run locally?

I'm not sure what you mean. I think this worked when I was testing changes for ghost arguments.

@dc-mak
Copy link
Contributor

dc-mak commented Jul 29, 2025

I'm not sure what you mean. I think this worked when I was testing changes for ghost arguments.

Opam only installs stuff which is committed, it ignores any unstaged changes. So if the user is relying on those, they'd have to temporarily commit and do opam install.

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.

2 participants