Skip to content

Conversation

puzzledpolymath
Copy link
Contributor

Thinking ahead when needing to support Snowflake's, in preparation for https://github.com/cycle/entity-behavior-identifier see 🎟️ cycle/entity-behavior-uuid#13

πŸ” What was changed

Added annotations and column mapping for each driver.

πŸ“ Checklist

  • How was this tested:
    • Unit tests added

Copy link

codecov bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests βœ…

Project coverage is 95.42%. Comparing base (876fbc2) to head (84347ae).
Report is 1 commits behind head on 2.x.

Additional details and impacted files
@@            Coverage Diff            @@
##                2.x     #235   +/-   ##
=========================================
  Coverage     95.42%   95.42%           
  Complexity     1896     1896           
=========================================
  Files           131      131           
  Lines          5285     5285           
=========================================
  Hits           5043     5043           
  Misses          242      242           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@puzzledpolymath puzzledpolymath changed the title Snowflake identifier Add Snowflake Identifier Jul 15, 2025
@puzzledpolymath
Copy link
Contributor Author

@roxblnfk Once merged and released, I'll be able to submit a PR to cycle/entity-behavior-identifier supporting Snowflake's

@roxblnfk roxblnfk merged commit 49a90c7 into 2.x Jul 22, 2025
32 checks passed
@roxblnfk roxblnfk deleted the snowflake-identifier branch July 22, 2025 04:37
@puzzledpolymath
Copy link
Contributor Author

@roxblnfk Thanks. When you get a moment would you mind taking a look at cycle/entity-behavior#40 which is the last PR blocking cycle/entity-behavior-identifier#5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants