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

feat: Pass Any URL That Includes the Build URL via --url Option #174

Merged
merged 4 commits into from
Dec 1, 2022

Conversation

ismet55555
Copy link
Owner

@ismet55555 ismet55555 commented Dec 1, 2022

Change Motivation

(A very brief summary on why you made these changes)

Ability to pass any URL that includes the build URL. Sometimes you just want to paste a build URL into the command like when viewing console logs for a build.

Based on contributors work: #173


Describe the Changes Made

(A very brief summary on what changes you made)

  • Ability to pass any type of build URL using the --url option
  • Example commands:
    • yojenkins build rebuild--url http://localhost:8080/job/my_folder/job/15/1/console
    • yojenkins build monitor --url http://localhost:8080/job/my_folder/job/my_configured_job/62/api/json/lsadjf/
  • Example URL:
    • Specified: http://localhost:8080/job/my_folder/job/my_configured_job/62/console
    • Interpreted: http://localhost:8080/job/my_folder/job/my_configured_job/62/
  • Examples of URLs that are valid:
    • http://localhost:8080/job/my_folder/job/my_configured_job/62/console
    • http://localhost:8080/job/my_folder/job/my_configured_job/62/something/blah
    • http://localhost:8080/job/my_folder/job/15/62/api/json

Change Type

(Check any that apply)

  • Bug Fix
  • New Feature
  • Code Refactoring
  • Documentation
  • Formatting
  • Automation
  • Unit Tests
  • Other

How Has This Been Tested?

(Please describe the tests/commands that you ran to verify your changes)

Locally using containerized Jenkins server


Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings

@ismet55555 ismet55555 added the enhancement New feature or request label Dec 1, 2022
@ismet55555 ismet55555 self-assigned this Dec 1, 2022
@ismet55555 ismet55555 merged commit e92efe7 into main Dec 1, 2022
@ismet55555 ismet55555 deleted the build-number-parse branch December 1, 2022 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants