Skip to content

Commit

Permalink
Merge pull request #352 from intersystems/document-pull-event-handlers
Browse files Browse the repository at this point in the history
Issue #350 documented pull event class
  • Loading branch information
isc-tleavitt authored Jun 7, 2024
2 parents 59a04d9 + c2c691e commit 83d71e8
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,12 @@ This might look like:
![Example of mapping configuration](docs/images/settings.PNG "Example of mapping configuration")
### Pull Event Handlers
The ##class(SourceControl.Git.PullEventHandler) is a base class that can be extended in order to develop functionality that should be run when the repository pulls from remote. The code placed inside the subclass' OnPull() method will be executed any time a pull occurs.
A recommended way to implement CI/CD would be to use one of the pre-defined subclasses of PullEventHandler that are placed inside the PullEventHandler package. Additionally, custom load logic can be placed in that package following the model of the existing subclasses.
### Security
#### Unsecured (http) connections
Expand Down Expand Up @@ -99,6 +105,8 @@ Assuming you have the local and remote repositories created,
`git config core.sshCommand 'ssh -i ~/.ssh/<private key name>'`
8. Test the refresh button for the remote branches on the WebUI, fetch from the source control menu in Studio or VS Code, and `git fetch` in Git Bash. All 3 should work without any issues.
## During Development
:warning: Whenever any code in this project is updated outside the server (e.g. after every `git pull`), you _have_ to run `zpm "load <absolute path to git-source-control>"`. Otherwise, the changes won't be reflected on the server. However, if you load git-source-control via the InterSystems package manager and run `git pull` via the extension itself with the default pull event handler configured, it'll just work.

0 comments on commit 83d71e8

Please sign in to comment.