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

Remove useless gitignore line #211

Merged
merged 1 commit into from
Dec 8, 2022
Merged

Remove useless gitignore line #211

merged 1 commit into from
Dec 8, 2022

Conversation

pojiro
Copy link
Contributor

@pojiro pojiro commented Dec 8, 2022

WHY: remove
リポジトリ内にできてしまったファイルは意図して無視するべきです。
例えばバグなどにより意図せず異なるパスに生成してしまったファイルを無視してしまうと、
バグに気づけなくなるという副作用があります。

WHY: remove
リポジトリ内にできてしまったファイルは意図して無視するべきです。
例えばバグなどにより意図せず異なるパスに生成してしまったファイルを無視してしまうと、
バグに気づけなくなるという副作用があります。
@pojiro pojiro marked this pull request as ready for review December 8, 2022 05:02
@takasehideki
Copy link
Member

ありがとうございます!
これは正確には,Makefile の awesome な改良によって不要になったといえます.v0.7.0以前(?)では NIF 関係の .o .so は平場に生成していましたが,以降の改善によって _build/ 配下に生成されるようになりました.ということで,merge します!!

@takasehideki takasehideki merged commit 29b76cc into main Dec 8, 2022
@takasehideki takasehideki deleted the hotfix-pojiro branch December 8, 2022 06:50
takasehideki added a commit that referenced this pull request Dec 12, 2022
**Full Changelog**: v0.8.2...v0.8.3

* New features: none
  * Add arm32v7 support to mix rclex.prep.ros2 by @pojiro in #210
* Code Improvements/Fixes:
  * improve doc about docker env by @takasehideki in #208
  * Remove useless gitignore line by @pojiro in #211
  * insert sleep before publishing on example code (see #212) by @takasehideki in #213
* Bumps:
  * Bump elixir_make from 0.7.0 to 0.7.1 by @dependabot in #209
* Known issues to be addressed in the near future:
  * `publish/2` sometimes failed just after `create_publisher/3` in #212
  * Lock `git_hooks` to 0.6.5 due to its issue in #138
  * Bump to Humble Hawksbill in #114
  * Release rcl nif resources when GerServer terminates in #160
* Note in this release: none
@takasehideki takasehideki mentioned this pull request Dec 12, 2022
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.

None yet

2 participants