Skip to content

Commit

Permalink
don't lower() the packit pr data before manging it
Browse files Browse the repository at this point in the history
packit coprs are case sensitive (it's Katello, not katello), but the
lower() was breaking that.

the original idea was that the lower() catches GitHub.COM and other
spellings, but it seems to do more harm than good, so let's remove it
  • Loading branch information
evgeni committed Feb 14, 2024
1 parent 9f2ec84 commit 1517f1a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion roles/packit/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
chroot: "rhel-{{ ansible_distribution_major_version }}-x86_64"
loop: "{{ packit_prs | default([]) }}"
vars:
packit_data: "{{ item.lower().replace('https://github.com/', '').replace('/pull', '').split('/') }}"
packit_data: "{{ item.replace('https://github.com/', '').replace('/pull', '').split('/') }}"
packit_org: "{{ packit_data[0] }}"
packit_repo: "{{ packit_data[1] }}"
packit_pr: "{{ packit_data[2] }}"

0 comments on commit 1517f1a

Please sign in to comment.