Skip to content
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

Switch to markdown cli action #1443

Merged
merged 2 commits into from
Jan 11, 2021

Conversation

Confectrician
Copy link
Contributor

Remove the current action after testing, sicne it doesn't fit my needs.
Testing markdown-cli as an action now, which had good results while testing local.

Signed-off-by: Jerome Luckenbach <[email protected]>
Signed-off-by: Jerome Luckenbach <[email protected]>
@netlify
Copy link

netlify bot commented Jan 11, 2021

✔️ Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).

🔨 Explore the source changes: 2c07860

🔍 Inspect the deploy logs: https://app.netlify.com/sites/openhab-docs-preview/deploys/5ffca1ae001a3b000727e70b

😎 Browse the preview: https://deploy-preview-1443--openhab-docs-preview.netlify.app

@Confectrician Confectrician merged commit a69863c into openhab:main Jan 11, 2021
ghys pushed a commit that referenced this pull request Jan 29, 2021
* Apply bulk automated corrections. 🔧(#1433)

Signed-off-by: Jerome Luckenbach <[email protected]>

* Added sidebar links. (#1434)

* Added sidebar links.
* Introduce openhabian placeholder page.
* Remove local version of openhabian doc page.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix linter errors 🔧(#1435)

* Fix errros for the addon folder.
* Fix errros for the administration folder.
* Fix errros for the concepts folder.
* Fix errros for the configuration folder.
* Improved and synced markdown linting rules.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Revert #1428 (#1436)

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix linter errors. 🔧(#1437)

Signed-off-by: Jerome Luckenbach <[email protected]>

* Bulk edit linter errors. (#1438)

* Bulk edit linter errors.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix linter style

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix linter errors (#1439)

* Some leftover linting error fixes.
* Adapt rules and enble markdownlint for pull requests.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix markdownlint workflow

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fixes for linter errors and markdown configuration. (#1440)

Signed-off-by: Jerome Luckenbach <[email protected]>

* Update guidelines.md (#1441)

Only a maintainer told me how to tackle a DTO properly as a class. As I did not see this anywhere, I decide to make this change.

* Add fixes from markdownlint-cli (#1442)

Signed-off-by: Jerome Luckenbach <[email protected]>

* Switch to markdown cli action (#1443)

* Fix left markdownlint errors.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Change to markdownlint-cli action.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Update markdownlint.yml

Signed-off-by: Jerome Luckenbach <[email protected]>

* Move trigger order

Signed-off-by: Jerome Luckenbach <[email protected]>

* Make pr check available for markdownlint 📝 (#1446)

* Make pr check available for markdownlint (hopefully).
Signed-off-by: Jerome Luckenbach <[email protected]>

* Add link check action. (#1449)

* Add link check action.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Add blank line.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Replace local urls with inline code. Change one occurence to openhabian. (#1450)

Signed-off-by: Jerome Luckenbach <[email protected]>

* remove uncessary instructions f. contribution guide (#1454)

* Update items.md (#1455)

Example with item Switch Kitchen_Light in the example is not correct for current binding version.

* Created API token docs (#1453)

* Created API token docs

Signed-off-by: Gerwin Lammers <[email protected]>

* Update after review session

Signed-off-by: Gerwin Lammers <[email protected]>

* Add new page to sidebar navigation.

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* Update rules-dsl.md (#1456)

* Update rules-dsl.md

Corrected the "system started" section to reflect that the trigger will no longer execute on loading the rules file in OH v.3.

* Adapt inline codeblock design.

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* Remove paper ui references. (#1457)

* Remove paper uii references.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Fix Markdown errors.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Remove target param from markdownlint pr trigger.

Signed-off-by: Jerome Luckenbach <[email protected]>

* Update items.md (#1458)

There is one comma too much in the sample for profile hysteresis. The profile must be related to the channel. The comma will lead also to an error message in log.
Switch Low_Battery { channel="serialbutton:button:mybutton:battery-level" [profile="hysteresis", lower=15, inverted=true] }

* Remove cronmaker since it doesn't support HTTPS (#1459)

1. When trying to reach https://www.cronmaker.com/ the website refuses the connection. You can only reach it using HTTP (without HTTPS). 
You could leave cronmaker in there but I think leading users to web pages that aren't using HTTPS might be not be best idea.

2. Fix Typo in url for quartz-scheduler.org

* Update links to target "main" branches (#1460)

* Update links to target "main" branches

Signed-off-by: Wouter Born <[email protected]>

* Update process_file.rb

Signed-off-by: Wouter Born <[email protected]>

* Update for all other repos too

Signed-off-by: Wouter Born <[email protected]>

* Add hint that /etc/default/openhab2 is not considered by the upgrade (#1461)

* Add hint that /etc/default/openhab2 is not considered by the upgrade 

When you use a raspberry you may have used the file /etc/default/openhab2 to changed JAVA options.
After upgrade to openHAB 3.0 this file is no longer used. Instead the file /etc/default/openhab must be used.

* Small modifications to original pull request.

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: Jerome Luckenbach <[email protected]>

* Fix typo in restdocs (#1463)

* Update rules-dsl.md (#1465)

Updated the rule example "Start wake up light on sunrise" to use the new formatting of the receivedEvent implicit variable

* Update list of system default channel types (#1467)

Related to openhab/openhab-core#2069
Related to openhab/openhab-core#2157

Signed-off-by: Christoph Weitkamp <[email protected]>

* Delete habpanel.md (#1471)

* Delete habpanel.md

This file is gathered from its code repository for a long time already.
Signed-off-by: Jerome Luckenbach <[email protected]>

* Add habpanel placeholder for preview

Signed-off-by: Jerome Luckenbach <[email protected]>

Co-authored-by: coop-git <[email protected]>
Co-authored-by: J-N-K <[email protected]>
Co-authored-by: sancho-sumy <[email protected]>
Co-authored-by: Gerwin Lammers <[email protected]>
Co-authored-by: dathbe <[email protected]>
Co-authored-by: thefechner <[email protected]>
Co-authored-by: Felix Schneider <[email protected]>
Co-authored-by: Wouter Born <[email protected]>
Co-authored-by: BertramVielsack <[email protected]>
Co-authored-by: mueller-ma <[email protected]>
Co-authored-by: lukiep111 <[email protected]>
Co-authored-by: Christoph Weitkamp <[email protected]>
@Confectrician Confectrician deleted the switch-to-markdown-cli branch December 8, 2021 22:54
@Confectrician Confectrician added this to the 3.1 milestone Nov 18, 2022
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.

1 participant