Skip to content

feat: Add ignored columns reporting#9

Merged
morganjezequel merged 6 commits intomasterfrom
report-unspecified-columns
Jun 9, 2025
Merged

feat: Add ignored columns reporting#9
morganjezequel merged 6 commits intomasterfrom
report-unspecified-columns

Conversation

@morganjezequel
Copy link
Contributor

This commit introduces a new feature that provides reporting of ignored columns when the corresponding option is enabled.

A new option, report_ignored_columns, has been added to the Sheetah::Template initialization. It works in conjunction with the existing ignore_unspecified_columns option, which must also be enabled for the reporting to take effect.

@morganjezequel morganjezequel force-pushed the report-unspecified-columns branch 4 times, most recently from f94427c to 1678b7f Compare May 15, 2025 14:38
This commit introduces a new feature that provides reporting
of ignored columns when the corresponding option is enabled.

A new option, `report_ignored_columns`, has been added to the
`Sheetah::Template` initialization. It works in conjunction
with the existing `ignore_unspecified_columns` option, which
must also be enabled for the reporting to take effect.
@morganjezequel morganjezequel force-pushed the report-unspecified-columns branch from 1678b7f to 83b4f4c Compare May 15, 2025 15:11
Morgan Jézéquel added 4 commits May 15, 2025 17:14
Updating Nokogiri is needed:

> nokogiri-1.15.4-x86_64-linux requires ruby version < 3.3.dev, >= 2.7
@morganjezequel morganjezequel force-pushed the report-unspecified-columns branch from f36b030 to 4b52a18 Compare May 16, 2025 07:21
Remove support for ruby 2.7, add support for ruby 3.3 and 3.4.
@morganjezequel morganjezequel force-pushed the report-unspecified-columns branch from 4b52a18 to f3957e1 Compare May 16, 2025 07:22
@morganjezequel morganjezequel merged commit f54c866 into master Jun 9, 2025
7 checks passed
@morganjezequel morganjezequel deleted the report-unspecified-columns branch June 9, 2025 07:11
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.

2 participants