Skip to content

Conversation

@Argmaster
Copy link
Owner

This pull request adds simple support for comment based attributes with G04 #@! <attr-type><attr-content> syntax. Such attributes are represented by default non-comment tokes, so parsing Gerber file with comment based attributes and dumping it from PyGerber AST representation will result in comment based attributes being transformed to normal attributes.

Resolves: #118

@Argmaster Argmaster added the enhancement New feature or request label Jun 15, 2024
@Argmaster Argmaster added this to the 2.5.0 milestone Jun 15, 2024
@Argmaster Argmaster self-assigned this Jun 15, 2024
@codecov
Copy link

codecov bot commented Jun 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.59%. Comparing base (909ab83) to head (080c9f0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   81.56%   81.59%   +0.02%     
==========================================
  Files         194      194              
  Lines        7671     7678       +7     
==========================================
+ Hits         6257     6265       +8     
+ Misses       1414     1413       -1     
Flag Coverage Δ
unittests 81.59% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Argmaster Argmaster merged commit 90c827e into main Jun 16, 2024
@Argmaster Argmaster deleted the feature/comment-based-attributes branch June 16, 2024 11:57
Argmaster added a commit that referenced this pull request Aug 11, 2024
This pull request adds simple support for comment based attributes with
`G04 #@! <attr-type><attr-content>` syntax. Such attributes are
represented by default non-comment tokes, so parsing Gerber file with
comment based attributes and dumping it from PyGerber AST representation
will result in comment based attributes being transformed to normal
attributes.

Resolves: #118
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add support for comment based attributes

2 participants