Skip to content

Commit

Permalink
Add 'context' to mergeDeep name fields
Browse files Browse the repository at this point in the history
This allows mergeDeep to correctly identify
changes to required_status_checks in rulesets

Signed-off-by: Kyle Harding <[email protected]>
  • Loading branch information
klutchell committed Oct 2, 2024
1 parent de7ade1 commit ac1dbc3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/mergeDeep.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
const mergeBy = require('./mergeArrayBy')
const DeploymentConfig = require('./deploymentConfig')

const NAME_FIELDS = ['name', 'username', 'actor_id', 'login', 'type', 'key_prefix']
const NAME_FIELDS = ['name', 'username', 'actor_id', 'login', 'type', 'key_prefix', 'context']
const NAME_USERNAME_PROPERTY = item => NAME_FIELDS.find(prop => Object.prototype.hasOwnProperty.call(item, prop))
const GET_NAME_USERNAME_PROPERTY = item => { if (NAME_USERNAME_PROPERTY(item)) return item[NAME_USERNAME_PROPERTY(item)] }

Expand Down

0 comments on commit ac1dbc3

Please sign in to comment.