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.

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

oj-verifyのインクルードパス #41

Open
chaemon opened this issue Sep 14, 2021 · 2 comments
Open

oj-verifyのインクルードパス #41

chaemon opened this issue Sep 14, 2021 · 2 comments

Comments

@chaemon
Copy link
Collaborator

chaemon commented Sep 14, 2021

oj-verifyでインクルードパスが指定できないことから、Nim-ACLのインクルードパスであるsrc/でoj-verifyを実行するか、src/以下をoj-verifyの実行場所に移動しなくてはなりません。

現状は後者でやっていますが、以下のような問題があります。

  1. 一部のテストが実行されない不具合がある。(原因はよくわかっていませんがmvをするとタイムスタンプが変わるところが怪しいです)
  2. 生成されたドキュメントのリンク先のgithubページに飛べない

前者の場合はsrcディレクトリに.verify-helperができてしまってソース以外のものがあるあたりが見栄えが悪いです。

どちらがいいですかね。

@zer0-star
Copy link
Owner

個人的には、前者の方でやっても良いという気持ちになってきました

実際にそうするとなると、ドキュメントの生成など、少し手を加える必要があるかもしれませんね。もしよければ、developブランチの方で試しにやってみようと思うのですが、どうでしょうか?

@chaemon
Copy link
Collaborator Author

chaemon commented Sep 17, 2021

はい!前者の方をdevelopブランチでやってみますか。

stableで生成されるドキュメントとdevelopで生成されるドキュメントのURLが異なっていて使い分けられると嬉しいですね!

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

No branches or pull requests

2 participants