Sitelet https://github.com/crankyoldgit/IRremoteESP8266/pull/2091
Skip to content

Create sk-SK.h - #2091

Merged
crankyoldgit merged 2 commits into
crankyoldgit:masterfrom
misa1515:patch-1
May 12, 2024
Merged

crankyoldgit merged 2 commits into
crankyoldgit:masterfrom
misa1515:patch-1

Conversation

@misa1515

Copy link
Copy Markdown
Contributor

Slovak language

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +4 to +5
#ifndef LOCALE_SK-SK_H_
#define LOCALE_SK-SK_H_

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As the build failure says, that dash - should be an underscore _...

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep.

src/locale/sk-SK.h:4:  #ifndef header guard has wrong style, please use: LOCALE_SK_SK_H_  [build/header_guard] [5]
src/locale/sk-SK.h:3[9](https://github.com/crankyoldgit/IRremoteESP8266/actions/runs/8870139943/job/24351598938?pr=2091#step:6:10)3:  #endif line should be "#endif  // LOCALE_SK_SK_H_"  [build/header_guard] [5]

Comment thread src/locale/sk-SK.h Outdated
"Tomuto výsledku by sa nemalo dôverovať, kým sa to nevyrieši. " \
"Upraviť a zväčšiť `kCaptureBufferSize`."

#endif // LOCALE_SK-SK_H_

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same error as at the top...

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src/locale/sk-SK.h:4:  #ifndef header guard has wrong style, please use: LOCALE_SK_SK_H_  [build/header_guard] [5]
src/locale/sk-SK.h:3[9](https://github.com/crankyoldgit/IRremoteESP8266/actions/runs/8870139943/job/24351598938?pr=2091#step:6:10)3:  #endif line should be "#endif  // LOCALE_SK_SK_H_"  [build/header_guard] [5]

@crankyoldgit crankyoldgit left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome. Thanks for adding this. Great effort! Just a few fixes and some stuff you should be able to remove.

Comment thread src/locale/sk-SK.h Outdated
"Tomuto výsledku by sa nemalo dôverovať, kým sa to nevyrieši. " \
"Upraviť a zväčšiť `kCaptureBufferSize`."

#endif // LOCALE_SK-SK_H_

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src/locale/sk-SK.h:4:  #ifndef header guard has wrong style, please use: LOCALE_SK_SK_H_  [build/header_guard] [5]
src/locale/sk-SK.h:3[9](https://github.com/crankyoldgit/IRremoteESP8266/actions/runs/8870139943/job/24351598938?pr=2091#step:6:10)3:  #endif line should be "#endif  // LOCALE_SK_SK_H_"  [build/header_guard] [5]

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +17 to +18
#define D_STR_TURBO "Turbo"
#define D_STR_SUPER "Super"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When they are the same as the default, please don't include them.
i.e. You should be able to remove these two lines.

Comment thread src/locale/sk-SK.h
#define D_STR_LIGHT "Svetlo"
#define D_STR_POWERFUL "Výkonný"
#define D_STR_QUIET "Tichý"
#define D_STR_ECONO "Econo"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto

Comment thread src/locale/sk-SK.h
#define D_STR_FILTER "Filter"
#define D_STR_3D "3D"
#define D_STR_CELSIUS "Celzia"
#define D_STR_FAHRENHEIT "Fahrenheit"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto. Remove

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +65 to +66
#define D_STR_START "Štart"
#define D_STR_STOP "Stop"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, remove these two.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Štart is not Start 😉

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops. Yeah, just remove Stop

Comment thread src/locale/sk-SK.h Outdated
#define D_STR_DISPLAY "Displej"
#define D_STR_WEEKLY "Týždenne"
#define D_STR_WEEKLYTIMER D_STR_WEEKLY " " D_STR_TIMER // Needs `D_STR_WEEKLY`!
#define D_STR_WIFI "WiFi"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto. Remove WiFi

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +123 to +126
#define D_STR_MAX "Max"
#define D_STR_MAXIMUM "Maximum"
#define D_STR_MIN "Min"
#define D_STR_MINIMUM "Minimum"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove these four.

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +197 to +198
#define D_STR_TRUE "True"
#define D_STR_FALSE "False"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove these two.

Comment thread src/locale/sk-SK.h Outdated
Comment on lines +202 to +226

// Model Names
#define D_STR_YAW1F "YAW1F"
#define D_STR_YBOFB "YBOFB"
#define D_STR_YX1FSF "YX1FSF"
#define D_STR_V9014557_A "V9014557-A"
#define D_STR_V9014557_B "V9014557-B"
#define D_STR_RLT0541HTA_A "R-LT0541-HTA-A"
#define D_STR_RLT0541HTA_B "R-LT0541-HTA-B"
#define D_STR_ARRAH2E "ARRAH2E"
#define D_STR_ARDB1 "ARDB1"
#define D_STR_ARREB1E "ARREB1E"
#define D_STR_ARJW2 "ARJW2"
#define D_STR_ARRY4 "ARRY4"
#define D_STR_ARREW4E "ARREW4E"
#define D_STR_GE6711AR2853M "GE6711AR2853M"
#define D_STR_AKB75215403 "AKB75215403"
#define D_STR_AKB74955603 "AKB74955603"
#define D_STR_AKB73757604 "AKB73757604"
#define D_STR_LG6711A20083V "LG6711A20083V"
#define D_STR_KKG9AC1 "KKG9AC1"
#define D_STR_KKG29AC1 "KKG29AC1"
#define D_STR_LKE "LKE"
#define D_STR_NKE "NKE"
#define D_STR_DKE "DKE"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should be able to remove all the model & protocol names.

@crankyoldgit crankyoldgit self-assigned this May 12, 2024
Review feedback corrections.
@crankyoldgit
crankyoldgit self-requested a review May 12, 2024 11:03
@crankyoldgit
crankyoldgit merged commit 32d10a6 into crankyoldgit:master May 12, 2024
crankyoldgit added a commit that referenced this pull request Jan 2, 2026
**[Bug Fixes]**
- Bosch: Fixed the bug where the wind speed was always set to auto. (#2237)
- Update IRsend sendHaierAC to include SEND_HAIER_AC160 (#2172)
- Gree: Fix reporting vertical swing (#2125)
- Fix `decodeYork()` parameter names & defaults. (#2121)
- Fix the Coolix fan-only mode in IRac class. (#2104)
- Fix missing quiet parameter of haier176 (#2102)
- ESP32-C3: Fix compilation error when USB CDC on Boot is enabled (#2080)

**[Features]**
- Add Fahrenheit support for the BOSCH144 protocol (#2224)
- Build: Add compatibility with C++20 (#2040)
- Add initial detailed support for Kelon168 (Kelon/Hisense) (#1949)
- Add support for the Eurom A/C protocol (#2208)
- Add Fahrenheit support for Coolix (#2214)
- ESP32: Esp32 Core version 3 support (#2144)
- auto_analyse_raw_data: Add kXxMsbFirst to easy change MSBFirst for the full protocol (#2143)
- change kAirtonMaxTemp from 25C to 31C (#2124)
- Added support for Bluestar Heavy AC (#2120)
- Add support of Toshiba Remote Control B (#2094)
- Update haier160 & HaierYRWO2 to use quiet in the common class. (#2115)
- Internationalisation: Solvakian translation (#2091)
- Daikin: Support setting temperature in 0.5 C unit (#2036)
- Quiet/Silent Mode for Electra_AC (#1990)

**[Misc]**
- Document Fischer R51L1/BGE remote support (#2231)
- CI: pin python v3.13
- CI: Attempt to fix intelhex failures
- IRMQTTServer: Fixes for ArduinoJson v7 to remove depreicated calls
- docs: updated contributing section for clarity (by Prerna Utage) (#2221)
- Fix typo in Russian language support (#2210)
- Build: Update CodeQL actions plugin to use v3 as v2 will be deprecated soon
- Build: Fix soon to be deprecated set-output command
- Build: Update build scripts to use non-deprecated actions tooling
- Fix linter issues (#2173)
- pylint fix raw_to_pronto_code.py (#2150)
- Document support for Comfee model (#2147)
- DAIKIN: ARC443A5 Remote supported note (#2138)
- library.json specifies libCompatMode strict (#2111)
- Added Electrolux EACM CL/N3 series remote to TCL protocol (#2100)
- Add AR-JW19 to supported devices (#2069)
- Remove unused constant `kRcmmExcess` (#2033)
- Panasonic AC: Document support for PV1122V remote (#2029)
- Document support for Panasonic CS-E12QKEW A/C (#2028)
@crankyoldgit crankyoldgit mentioned this pull request Jan 2, 2026
crankyoldgit added a commit that referenced this pull request Jan 2, 2026
**[Bug Fixes]**
- Bosch: Fixed the bug where the wind speed was always set to auto. (#2237)
- Update IRsend sendHaierAC to include SEND_HAIER_AC160 (#2172)
- Gree: Fix reporting vertical swing (#2125)
- Fix `decodeYork()` parameter names & defaults. (#2121)
- Fix the Coolix fan-only mode in IRac class. (#2104)
- Fix missing quiet parameter of haier176 (#2102)
- ESP32-C3: Fix compilation error when USB CDC on Boot is enabled (#2080)

**[Features]**
- Add Fahrenheit support for the BOSCH144 protocol (#2224)
- Build: Add compatibility with C++20 (#2040)
- Add initial detailed support for Kelon168 (Kelon/Hisense) (#1949)
- Add support for the Eurom A/C protocol (#2208)
- Add Fahrenheit support for Coolix (#2214)
- ESP32: Esp32 Core version 3 support (#2144)
- auto_analyse_raw_data: Add kXxMsbFirst to easy change MSBFirst for the full protocol (#2143)
- change kAirtonMaxTemp from 25C to 31C (#2124)
- Added support for Bluestar Heavy AC (#2120)
- Add support of Toshiba Remote Control B (#2094)
- Update haier160 & HaierYRWO2 to use quiet in the common class. (#2115)
- Internationalisation: Solvakian translation (#2091)
- Daikin: Support setting temperature in 0.5 C unit (#2036)
- Quiet/Silent Mode for Electra_AC (#1990)

**[Misc]**
- Document Fischer R51L1/BGE remote support (#2231)
- CI: pin python v3.13
- CI: Attempt to fix intelhex failures
- IRMQTTServer: Fixes for ArduinoJson v7 to remove depreicated calls
- docs: updated contributing section for clarity (by Prerna Utage) (#2221)
- Fix typo in Russian language support (#2210)
- Build: Update CodeQL actions plugin to use v3 as v2 will be deprecated soon
- Build: Fix soon to be deprecated set-output command
- Build: Update build scripts to use non-deprecated actions tooling
- Fix linter issues (#2173)
- pylint fix raw_to_pronto_code.py (#2150)
- Document support for Comfee model (#2147)
- DAIKIN: ARC443A5 Remote supported note (#2138)
- library.json specifies libCompatMode strict (#2111)
- Added Electrolux EACM CL/N3 series remote to TCL protocol (#2100)
- Add AR-JW19 to supported devices (#2069)
- Remove unused constant `kRcmmExcess` (#2033)
- Panasonic AC: Document support for PV1122V remote (#2029)
- Document support for Panasonic CS-E12QKEW A/C (#2028)
@crankyoldgit crankyoldgit mentioned this pull request Jan 2, 2026
crankyoldgit added a commit that referenced this pull request Jan 2, 2026
_v2.9.0 (20260103)_ release

**[Bug Fixes]**
- Bosch: Fixed the bug where the wind speed was always set to auto. (#2237)
- Update IRsend sendHaierAC to include SEND_HAIER_AC160 (#2172)
- Gree: Fix reporting vertical swing (#2125)
- Fix `decodeYork()` parameter names & defaults. (#2121)
- Fix the Coolix fan-only mode in IRac class. (#2104)
- Fix missing quiet parameter of haier176 (#2102)
- ESP32-C3: Fix compilation error when USB CDC on Boot is enabled (#2080)

**[Features]**
- Add Fahrenheit support for the BOSCH144 protocol (#2224)
- Build: Add compatibility with C++20 (#2040)
- Add initial detailed support for Kelon168 (Kelon/Hisense) (#1949)
- Add support for the Eurom A/C protocol (#2208)
- Add Fahrenheit support for Coolix (#2214)
- ESP32: Esp32 Core version 3 support (#2144)
- auto_analyse_raw_data: Add kXxMsbFirst to easy change MSBFirst for the full protocol (#2143)
- change kAirtonMaxTemp from 25C to 31C (#2124)
- Added support for Bluestar Heavy AC (#2120)
- Add support of Toshiba Remote Control B (#2094)
- Update haier160 & HaierYRWO2 to use quiet in the common class. (#2115)
- Internationalisation: Solvakian translation (#2091)
- Daikin: Support setting temperature in 0.5 C unit (#2036)
- Quiet/Silent Mode for Electra_AC (#1990)

**[Misc]**
- Document Fischer R51L1/BGE remote support (#2231)
- CI: pin python v3.13
- CI: Attempt to fix intelhex failures
- IRMQTTServer: Fixes for ArduinoJson v7 to remove depreicated calls
- docs: updated contributing section for clarity (by Prerna Utage) (#2221)
- Fix typo in Russian language support (#2210)
- Build: Update CodeQL actions plugin to use v3 as v2 will be deprecated soon
- Build: Fix soon to be deprecated set-output command
- Build: Update build scripts to use non-deprecated actions tooling
- Fix linter issues (#2173)
- pylint fix raw_to_pronto_code.py (#2150)
- Document support for Comfee model (#2147)
- DAIKIN: ARC443A5 Remote supported note (#2138)
- library.json specifies libCompatMode strict (#2111)
- Added Electrolux EACM CL/N3 series remote to TCL protocol (#2100)
- Add AR-JW19 to supported devices (#2069)
- Remove unused constant `kRcmmExcess` (#2033)
- Panasonic AC: Document support for PV1122V remote (#2029)
- Document support for Panasonic CS-E12QKEW A/C (#2028)
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.

3 participants