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.

Sign up for GitHub

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

Jump to bottom

Add Linux build CI #43

Merged
merged 7 commits into from
Jan 27, 2022
Merged

Add Linux build CI #43

merged 7 commits into from
Jan 27, 2022

Conversation

sksat
Copy link
Collaborator

@sksat sksat commented Jan 26, 2022 •
edited
Loading

Overview

  • Add Linux support for ExtLibraries CMake
  • Add Linux build CI

Issue/PR

Details

Write in detail.

Validation results

Link to tests or validation results.

Scope of influence

eg. The behavior of XX will be change.

Supplement

Write additional comments if you need.

Note

  • If there are related Projects, tie them together.
  • Assignees should be set if possible.
  • Reviewers should be set if possible.
  • Set priority label if possible.

@sksat sksat added the enhancement New feature or request label Jan 26, 2022
@sksat sksat self-assigned this Jan 26, 2022
@sksat
Copy link
Collaborator Author

sksat commented Jan 26, 2022

さてCIがコケたがこれは僕のせいではなく，なんとこのタイミングでSpaceWeather-v1.2.txtが更新されたから．

DATATYPE CssiSpaceWeather
VERSION 1.2 
UPDATED 2022 Jan 26 09:40:00.00

@sksat
Copy link
Collaborator Author

sksat commented Jan 26, 2022

SpaceWeather-v1.2.txtはあくまでデータだし，シミュレーションをCIで回す時にまた考えるとしてチェックサム見なくていい説もある(ファイル名にバージョン書いてあるし...(これはフォーマットのバージョン？))

@sksat
Copy link
Collaborator Author

sksat commented Jan 26, 2022

よしできた

@sksat sksat changed the title Add Linux build Add Linux build CI Jan 26, 2022
@sksat sksat requested a review from 200km January 26, 2022 12:15
@sksat
Copy link
Collaborator Author

sksat commented Jan 27, 2022

表示が分かりにくいなと思ったのでjob名だけ足しました

meltingrabbit
meltingrabbit approved these changes Jan 27, 2022
View reviewed changes
Copy link
Contributor

@meltingrabbit meltingrabbit left a comment

Choose a reason for hiding this comment

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

だいたい理解したのでapprove

@sksat sksat merged commit 4580e3d into develop Jan 27, 2022
@sksat sksat deleted the feature/linux-build branch January 27, 2022 05:43
@sksat sksat mentioned this pull request Apr 18, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@meltingrabbit meltingrabbit meltingrabbit approved these changes

@200km 200km Awaiting requested review from 200km

Assignees

@sksat sksat

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sksat @meltingrabbit