ansible-collections / community.proxysql

ProxySQL Ansible Collection
http://galaxy.ansible.com/community/proxysql
GNU General Public License v3.0
21 stars 26 forks source link

fix ci: too many blank lines #120

Closed markuman closed 2 years ago

markuman commented 2 years ago

ERROR: Found 1 yamllint issue(s) which need to be resolved: ERROR: tests/integration/targets/test_proxysql_query_rules_fast_routing/tasks/107-update_destination_hostgroup.yml:62:1: empty-lines: too many blank lines (1 > 0)

codecov[bot] commented 2 years ago

Codecov Report

Merging #120 (86b282a) into main (beca67f) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #120   +/-   ##
=======================================
  Coverage   75.14%   75.14%           
=======================================
  Files          10       10           
  Lines        1199     1199           
  Branches      193      193           
=======================================
  Hits          901      901           
  Misses        221      221           
  Partials       77       77           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Andersson007 commented 1 year ago

@markuman btw we can set up backporting by the bot here (using labels like stable-1 for example), we need to

  1. add a similar file https://github.com/ansible-collections/community.postgresql/blob/main/.github/patchback.yml
  2. install "Patchback" app to the repo (i can)

Just a fyi, if sounds OK, feel free to add the file and let me know when i can install the app

markuman commented 1 year ago

@markuman btw we can set up backporting by the bot here (using labels like stable-1 for example), we need to

1. add a similar file https://github.com/ansible-collections/community.postgresql/blob/main/.github/patchback.yml

2. install "Patchback" app to the repo (i can)

Just a fyi, if sounds OK, feel free to add the file and let me know when i can install the app

@Andersson007 sounds good. but the current backports must be still done manually.

Andersson007 commented 1 year ago

@markuman I've just enabled the app, so please: