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

Prepare Bazel for Pigweed #88

Merged
merged 5 commits into from
Aug 18, 2024
Merged

Prepare Bazel for Pigweed #88

merged 5 commits into from
Aug 18, 2024

Conversation

michael-christen
Copy link
Owner

@michael-christen michael-christen commented Aug 17, 2024

###Large Changes

Miscellaneous Changes

  • Updated .bazelignore to ignore bazel-bin, bazel-out, bazel-testlogs
  • add -diff to .gitattributes for our big lock files so when diffing we
    just see it as a "binary" and don't make git diff unintelligible
  • Fixed location of try-import in .bazelrc (should always be last)

NOTE: You can add per_file_copts with --per_file_copt, for example to
ignore warnings in third_party code

Followup

  • add support for stm32 and rp platforms

Related Issues:

Copy link

codecov bot commented Aug 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (8bac5f9) to head (e21eaf6).

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #88   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         5    -3     
  Lines           82        54   -28     
=========================================
- Hits            82        54   -28     

see 7 files with indirect coverage changes

@michael-christen michael-christen merged commit 6136cd7 into master Aug 18, 2024
5 checks passed
@michael-christen michael-christen deleted the pigweed_minimal branch August 18, 2024 15:12
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.

1 participant