FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[MINOR] Add support for the remainder of .gitmodules attributes by aureleon · Pull Request #26 · ValentinFrancois/python-gitlab-submodule · GitHub

[MINOR] Add support for the remainder of .gitmodules attributes - #26

Closed
aureleon wants to merge 1 commit into
ValentinFrancois:mainfrom
aureleon:extension/submodule-attributes
Closed

[MINOR] Add support for the remainder of .gitmodules attributes#26
aureleon wants to merge 1 commit into
ValentinFrancois:mainfrom
aureleon:extension/submodule-attributes

Conversation

Copy link
Copy Markdown
  • Adds all remaining .gitmodules "submodule" attributes to objects.Submodule: update, branch, ignore, shallow, fetchRecurseSubmodules (keyed as recurse)

  • Modifies read_gitmodules._read_gitmodules_file_content to use configparser instead of regex to simplify matching optional keys

  • Modifies any tests requiring direct Submodule instantiation, and adds a few extra arguments to test the extension

- Adds all remaining .gitmodules "submodule" attributes to objects.Submodule:
  update, branch, ignore, shallow, fetchRecurseSubmodules (keyed as recurse)

- Modifies read_gitmodules._read_gitmodules_file_content to use
  configparser instead of regex to simplify matching optional keys

- Modifies any tests requiring direct Submodule instantiation,
  and adds a few extra arguments to test the extension

aureleon commented Jun 10, 2024
edited
Loading

Copy link
Copy Markdown
Author

Came across this repository while perusing for a package to extend python-gitlab to include submodules. Needed access to other attributes like "submodule.[name].branch" and "submodule.[name].ignore", so I thought this would be a useful extension here.

Copy link
Copy Markdown
Owner

Hi, thanks for contributing!
Sorry for the late answer, I just fixed the outdated workflows so that they can run again on PRs.
The code lint check is failing, let me know if you want to fix it or if I should take over.
I will happily merge once the checks are all green.

Copy link
Copy Markdown
Owner

@aureleon I went ahead and merged your changes with lint fixes here, thanks again!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL