Skip to content

Add a first version for interfaces (non prob for now)#7

Draft
forrazh wants to merge 23 commits into
mainfrom
ping/interfaces
Draft

Add a first version for interfaces (non prob for now)#7
forrazh wants to merge 23 commits into
mainfrom
ping/interfaces

Conversation

@forrazh

@forrazh forrazh commented Jul 9, 2026

Copy link
Copy Markdown
Owner

No description provided.

@forrazh forrazh requested a review from affeldt-aist July 9, 2026 07:36
@forrazh forrazh self-assigned this Jul 9, 2026
Comment thread Makefile.rocq.conf Outdated
Comment thread Makefile.rocq Outdated
Comment thread .Makefile.rocq.d Outdated
@affeldt-aist

Copy link
Copy Markdown
Collaborator

Can't compile immediately:

 make
/Library/Developer/CommandLineTools/usr/bin/make -f Makefile.rocq all
W: This Makefile was generated by Rocq/Coq 9.0.0
W: while the current Rocq version is 9.0.1
make[2]: *** No rule to make target `/var/home/forrazh/code/phd/FreerDPS/_opam/lib/rocq-runtime/rocqworker', needed by `theories/Init.vo'.  Stop.
make[1]: *** [all] Error 2
make: *** [all] Error 2

This is because of the automatically generated files that have been committed.

@affeldt-aist affeldt-aist changed the title Add a first version for interfaces (non prob for nowrk Add a first version for interfaces (non prob for now) Jul 10, 2026
@affeldt-aist

affeldt-aist commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

It might be a good idea to follow MathComp's rules in terms of indentation and line width (and the absence of trailing spaces).

@forrazh forrazh force-pushed the ping/interfaces branch 2 times, most recently from 17f814b to bf9e46b Compare July 13, 2026 02:03
@forrazh

forrazh commented Jul 13, 2026

Copy link
Copy Markdown
Owner Author

I fixed the generated files issue on master and propagated (rebase) on my three PRs.
I'll look for the mathcomp "linting" rules

@forrazh

forrazh commented Jul 15, 2026

Copy link
Copy Markdown
Owner Author

Depends on #8

@forrazh forrazh marked this pull request as draft July 15, 2026 06:53
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