-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
MINOR: exclude all src/generated
and src/generated-test
#10671
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @chia7712 . Can you verify this by creating some test directories/file that follow this pattern, and make sure they don't show up? I had a vague impression that .gitignore files don't allow regular regexes and followed some niche syntax instead, but I'm not sure.
I'll leave it up to you to test and then merge if successful
the gitignore documentation (https://git-scm.com/docs/gitignore) has following example: I add file to |
…e-allocations-lz4 * apache-github/trunk: (155 commits) KAFKA-12728: Upgrade gradle to 7.0.2 and shadow to 7.0.0 (apache#10606) KAFKA-12778: Fix QuorumController request timeouts and electLeaders (apache#10688) KAFKA-12754: Improve endOffsets for TaskMetadata (apache#10634) Rework on KAFKA-3968: fsync the parent directory of a segment file when the file is created (apache#10680) MINOR: set replication.factor to 1 to make StreamsBrokerCompatibilityService work with old broker (apache#10673) MINOR: prevent cleanup() from being called while Streams is still shutting down (apache#10666) KAFKA-8326: Introduce List Serde (apache#6592) KAFKA-12697: Add Global Topic and Partition count metrics to the Quorum Controller (apache#10679) KAFKA-12648: MINOR - Add TopologyMetadata.Subtopology class for subtopology metadata (apache#10676) MINOR: Update jacoco to 0.8.7 for JDK 16 support (apache#10654) MINOR: exclude all `src/generated` and `src/generated-test` (apache#10671) KAFKA-12772: Move all transaction state transition rules into their states (apache#10667) KAFKA-12758 Added `server-common` module to have server side common classes. (apache#10638) MINOR Removed copying storage libraries specifically as they are already copied. (apache#10647) KAFKA-5876: KIP-216 Part 4, Apply InvalidStateStorePartitionException for Interactive Queries (apache#10657) KAFKA-12747: Fix flakiness in shouldReturnUUIDsWithStringPrefix (apache#10643) MINOR: remove unnecessary placeholder from WorkerSourceTask#recordSent (apache#10659) MINOR: Remove unused `scalatest` definition from `dependencies.gradle` (apache#10655) MINOR: checkstyle version upgrade: 8.20 -> 8.36.2 (apache#10656) KAFKA-12464: minor code cleanup and additional logging in constrained sticky assignment (apache#10645) ...
…0671) Reviewers: Anna Sophie Blee-Goldman <[email protected]>
…0671) Reviewers: Anna Sophie Blee-Goldman <[email protected]>
…0671) Summary: author: Chia-Ping Tsai <[email protected]> on 12/05/21 at 12:57 pm apache-reviewers: Anna Sophie Blee-Goldman <[email protected]> (cherry picked from commit 9c9ea30) Reviewers: #ldap_kafka_admins, satishd Reviewed By: #ldap_kafka_admins, satishd JIRA Issues: DKAFC-868 Differential Revision: https://code.uberinternal.com/D6303249
…0671) Summary: author: Chia-Ping Tsai <[email protected]> on 12/05/21 at 12:57 pm apache-reviewers: Anna Sophie Blee-Goldman <[email protected]> (cherry picked from commit 9c9ea30) Reviewers: #ldap_kafka_admins, satishd Reviewed By: #ldap_kafka_admins, satishd JIRA Issues: DKAFC-868 Differential Revision: https://code.uberinternal.com/D6303249
for #10637 (comment)
Committer Checklist (excluded from commit message)