Merge pull request #198 from crazy-max/refactor-default-banch

don't handle default branch global exp for pr events

README.md

@@ -693,9 +693,6 @@ workflow run. Will be empty for a branch reference:
 Returns `true` if the branch that triggered the workflow run is the default
 one, otherwise `false`.
 
-Will compare against the branch name that triggered the workflow run or the 
-base ref for a pull request.
-
 #### `{{date '<format>'}}`
 
 Returns the current date rendered by its [moment format](https://momentjs.com/docs/#/displaying/format/).

__tests__/meta.test.ts

@@ -2327,14 +2327,11 @@ describe('pr', () => {
       } as Inputs,
       {
         main: 'mytag-master',
-        partial: [
-          'mytag-defbranch'
-        ],
+        partial: [],
         latest: false
       } as Version,
       [
-        'org/app:mytag-master',
-        'org/app:mytag-defbranch'
+        'org/app:mytag-master'
       ],
       [
         "org.opencontainers.image.title=Hello-World",
@@ -2359,14 +2356,11 @@ describe('pr', () => {
       } as Inputs,
       {
         main: 'mytag-master',
-        partial: [
-          'mytag-defbranch'
-        ],
+        partial: [],
         latest: false
       } as Version,
       [
-        'org/app:mytag-master',
-        'org/app:mytag-defbranch'
+        'org/app:mytag-master'
       ],
       [
         "org.opencontainers.image.title=Hello-World",

src/meta.ts

@@ -377,17 +377,14 @@ export class Meta {
         return '';
       },
       is_default_branch: function () {
-        let branch = ctx.ref.replace(/^refs\/heads\//g, '');
+        const branch = ctx.ref.replace(/^refs\/heads\//g, '');
         // TODO: "base_ref" is available in the push payload but doesn't always seem to
         //  return the expected branch when the push tag event occurs. It's also not
         //  documented in GitHub docs: https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#push
         //  more context: https://github.com/docker/metadata-action/pull/192#discussion_r854673012
         // if (/^refs\/tags\//.test(ctx.ref) && ctx.payload?.base_ref != undefined) {
         //   branch = ctx.payload.base_ref.replace(/^refs\/heads\//g, '');
         // }
-        if (/^refs\/pull\//.test(ctx.ref) && ctx.payload?.pull_request?.base?.ref != undefined) {
-          branch = ctx.payload.pull_request.base.ref;
-        }
         if (branch == undefined || branch.length == 0) {
           return 'false';
         }