Skip to content

Remove branch restriction from build workflow pull_request trigger - #8

Merged
JamesDawson merged 2 commits into
mainfrom
copilot/update-build-workflow-trigger
Nov 13, 2025
Merged

Remove branch restriction from build workflow pull_request trigger#8
JamesDawson merged 2 commits into
mainfrom
copilot/update-build-workflow-trigger

Conversation

Copilot AI commented Nov 13, 2025

Copy link
Copy Markdown

The build workflow was only running on pull requests targeting main, preventing CI validation for PRs targeting other branches (e.g., feature branches, release branches).

Changes:

  • Removed branches filter from pull_request trigger in .github/workflows/build.yml

Before:

pull_request:
  branches:
  - main

After:

pull_request:

Build workflow now triggers for all pull requests regardless of target branch.

View original Slack conversation


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: JamesDawson <167181+JamesDawson@users.noreply.github.com>
Copilot AI changed the title [WIP] Update build workflow to trigger for all pull requests Remove branch restriction from build workflow pull_request trigger Nov 13, 2025
Copilot AI requested a review from JamesDawson November 13, 2025 12:42
@github-actions

Copy link
Copy Markdown

Code Coverage Summary Report - Linux (No TFM)

Summary
Generated on: 11/13/2025 - 12:49:22
Parser: Cobertura
Assemblies: 2
Classes: 201
Files: 452
Line coverage: 31.5% (10507 of 33299)
Covered lines: 10507
Uncovered lines: 22792
Coverable lines: 33299
Total lines: 53346
Branch coverage: 27.9% (2906 of 10398)
Covered branches: 2906
Total branches: 10398
Method coverage: Feature is only available for sponsors

Coverage

Endjin.FreeAgent.Client - 31.3%
Name Line Branch
Endjin.FreeAgent.Client 31.3% 27.7%
Endjin.FreeAgent.Client.AgedDebtorsAndCreditors 12.5% 7.1%
Endjin.FreeAgent.Client.Attachments 13.3% 0%
Endjin.FreeAgent.Client.BalanceSheetReports 22.2% 12.5%
Endjin.FreeAgent.Client.BankAccounts 100% 65%
Endjin.FreeAgent.Client.BankStatementUploads 6.2% 5.5%
Endjin.FreeAgent.Client.BankTransactionExplanations 11.1% 0%
Endjin.FreeAgent.Client.BankTransactions 8.2% 0%
Endjin.FreeAgent.Client.Bills 100% 62.5%
Endjin.FreeAgent.Client.CapitalAssets 5.4% 6.6%
Endjin.FreeAgent.Client.CapitalAssetTypes 26.6% 20%
Endjin.FreeAgent.Client.CashFlowReports 25% 20%
Endjin.FreeAgent.Client.Categories 33.3% 0%
Endjin.FreeAgent.Client.ClientBase 67.6% 28.2%
Endjin.FreeAgent.Client.ClientBase 67.6% 28.2%
Endjin.FreeAgent.Client.Company 23% 0%
Endjin.FreeAgent.Client.Contacts 100% 78.5%
Endjin.FreeAgent.Client.CorporationTaxReturns 8.6% 10%
Endjin.FreeAgent.Client.CreditNotes 74.6% 39.2%
Endjin.FreeAgent.Client.Currencies 26.6% 20%
Endjin.FreeAgent.Client.DepreciationProfiles 26.6% 20%
Endjin.FreeAgent.Client.Estimates 54.5% 40.2%
Endjin.FreeAgent.Client.Expenses 100% 50%
Endjin.FreeAgent.Client.FreeAgentClient 100% 50%
Endjin.FreeAgent.Client.FreeAgentClientServiceCollectionExtensions 0% 0%
Endjin.FreeAgent.Client.FreeAgentJsonContext 30.8% 28.5%
Endjin.FreeAgent.Client.FreeAgentOptions 100% 100%
Endjin.FreeAgent.Client.FreeAgentOptionsValidator 100% 100%
Endjin.FreeAgent.Client.HttpResponseMessageExtensions 42.8%
Endjin.FreeAgent.Client.Invoices 96% 52.9%
Endjin.FreeAgent.Client.JournalSets 100%
Endjin.FreeAgent.Client.Mileages 5.7% 5.2%
Endjin.FreeAgent.Client.Notes 38.4% 0%
Endjin.FreeAgent.Client.OAuth2.OAuth2Options 0% 0%
Endjin.FreeAgent.Client.OAuth2.OAuth2Service 0% 0%
Endjin.FreeAgent.Client.OAuth2.OAuth2Service.TokenInfo 0%
Endjin.FreeAgent.Client.OpeningBalances 14.8% 14.2%
Endjin.FreeAgent.Client.Payroll 5.8% 5.2%
Endjin.FreeAgent.Client.PayrollProfiles 7.2% 8.3%
Endjin.FreeAgent.Client.Payslips 10.2% 4.7%
Endjin.FreeAgent.Client.ProfitAndLossReports 22.2% 20%
Endjin.FreeAgent.Client.Projects 39.4% 20.4%
Endjin.FreeAgent.Client.RecurringInvoices 5% 6.2%
Endjin.FreeAgent.Client.SalesTaxRates 26.6% 20%
Endjin.FreeAgent.Client.SelfAssessmentReturns 7.8% 8.3%
Endjin.FreeAgent.Client.SharedJsonOptions 95.6%
Endjin.FreeAgent.Client.StockItems 6.8% 8.3%
Endjin.FreeAgent.Client.Tasks 75.8% 61.1%
Endjin.FreeAgent.Client.Timeslips 100% 83.3%
Endjin.FreeAgent.Client.TrialBalances 20% 12.5%
Endjin.FreeAgent.Client.Users 61.1% 68.4%
Endjin.FreeAgent.Client.VatReturns 100% 68.7%
Endjin.FreeAgent.Client.Webhooks 6.5% 8.3%
System.Text.RegularExpressions.Generated 0% 0%
System.Text.RegularExpressions.Generated.RunnerFactory
System.Text.RegularExpressions.Generated.RunnerFactory.Runner
Endjin.FreeAgent.Domain - 38.8%
Name Line Branch
Endjin.FreeAgent.Domain 38.8% 60.7%
Endjin.FreeAgent.Converters.RoleJsonConverter 74.2% 62.9%
Endjin.FreeAgent.Converters.RoleNonNullableJsonConverter 0% 0%
Endjin.FreeAgent.Domain.AgedCreditorEntry 0%
Endjin.FreeAgent.Domain.AgedDebtorEntry 0%
Endjin.FreeAgent.Domain.Attachment 0%
Endjin.FreeAgent.Domain.AttachmentRoot 0%
Endjin.FreeAgent.Domain.BalanceSheet 0%
Endjin.FreeAgent.Domain.BalanceSheetEntry 0%
Endjin.FreeAgent.Domain.BalanceSheetRoot 0%
Endjin.FreeAgent.Domain.BankAccount 95.2%
Endjin.FreeAgent.Domain.BankAccountDetails 0%
Endjin.FreeAgent.Domain.BankAccountRoot 50%
Endjin.FreeAgent.Domain.BankAccountsRoot 50%
Endjin.FreeAgent.Domain.BankStatementUpload 0%
Endjin.FreeAgent.Domain.BankStatementUploadResponse 0%
Endjin.FreeAgent.Domain.BankStatementUploadResponseRoot 0%
Endjin.FreeAgent.Domain.BankStatementUploadRoot 0%
Endjin.FreeAgent.Domain.BankTransaction 0%
Endjin.FreeAgent.Domain.BankTransactionExplanation 0%
Endjin.FreeAgent.Domain.BankTransactionExplanationRoot 0%
Endjin.FreeAgent.Domain.BankTransactionExplanationsRoot 0%
Endjin.FreeAgent.Domain.BankTransactionRoot 0%
Endjin.FreeAgent.Domain.BankTransactionsRoot 0%
Endjin.FreeAgent.Domain.Bill 95.4%
Endjin.FreeAgent.Domain.BillPayment 66.6%
Endjin.FreeAgent.Domain.BillPaymentRoot 50%
Endjin.FreeAgent.Domain.BillRoot 50%
Endjin.FreeAgent.Domain.BillsRoot 50%
Endjin.FreeAgent.Domain.CapitalAsset 0%
Endjin.FreeAgent.Domain.CapitalAssetRoot 0%
Endjin.FreeAgent.Domain.CapitalAssetsRoot 0%
Endjin.FreeAgent.Domain.CapitalAssetType 0%
Endjin.FreeAgent.Domain.CapitalAssetTypeRoot 0%
Endjin.FreeAgent.Domain.CapitalAssetTypesRoot 0%
Endjin.FreeAgent.Domain.CashFlow 0%
Endjin.FreeAgent.Domain.CashFlowItem 0%
Endjin.FreeAgent.Domain.CashFlowRoot 0%
Endjin.FreeAgent.Domain.CategoriesRoot 0%
Endjin.FreeAgent.Domain.Category 0%
Endjin.FreeAgent.Domain.Company 0%
Endjin.FreeAgent.Domain.CompanyRoot 0%
Endjin.FreeAgent.Domain.Contact 100%
Endjin.FreeAgent.Domain.ContactRoot 100%
Endjin.FreeAgent.Domain.ContactsRoot 50%
Endjin.FreeAgent.Domain.CorporationTaxReturn 0%
Endjin.FreeAgent.Domain.CorporationTaxReturnFiling 0%
Endjin.FreeAgent.Domain.CorporationTaxReturnFilingRoot 0%
Endjin.FreeAgent.Domain.CorporationTaxReturnRoot 0%
Endjin.FreeAgent.Domain.CorporationTaxReturnsRoot 0%
Endjin.FreeAgent.Domain.CreditNote 94.4%
Endjin.FreeAgent.Domain.CreditNoteEmailRoot 50%
Endjin.FreeAgent.Domain.CreditNoteEmailWrapper 50%
Endjin.FreeAgent.Domain.CreditNoteItem 0%
Endjin.FreeAgent.Domain.CreditNoteRefund 66.6%
Endjin.FreeAgent.Domain.CreditNoteRefundRoot 50%
Endjin.FreeAgent.Domain.CreditNoteRoot 50%
Endjin.FreeAgent.Domain.CreditNotesRoot 50%
Endjin.FreeAgent.Domain.CurrenciesRoot 0%
Endjin.FreeAgent.Domain.Currency 0%
Endjin.FreeAgent.Domain.DepreciationProfile 0%
Endjin.FreeAgent.Domain.DepreciationProfilesRoot 0%
Endjin.FreeAgent.Domain.Estimate 86.3%
Endjin.FreeAgent.Domain.EstimateItem 75%
Endjin.FreeAgent.Domain.EstimateRoot 50%
Endjin.FreeAgent.Domain.EstimatesRoot 50%
Endjin.FreeAgent.Domain.Expense 96%
Endjin.FreeAgent.Domain.ExpenseAttachment 80%
Endjin.FreeAgent.Domain.ExpenseRoot 50%
Endjin.FreeAgent.Domain.FreeAgentDateTimeFormatExtensions 0%
Endjin.FreeAgent.Domain.Invoice 97.2%
Endjin.FreeAgent.Domain.InvoiceEmail 85.7%
Endjin.FreeAgent.Domain.InvoiceEmailRoot 50%
Endjin.FreeAgent.Domain.InvoiceEmailWrapper 50%
Endjin.FreeAgent.Domain.InvoiceItem 93.7%
Endjin.FreeAgent.Domain.InvoicePayment 66.6%
Endjin.FreeAgent.Domain.InvoicePaymentRoot 50%
Endjin.FreeAgent.Domain.InvoiceRoot 50%
Endjin.FreeAgent.Domain.InvoicesRoot 50%
Endjin.FreeAgent.Domain.JournalEntry 85.7%
Endjin.FreeAgent.Domain.JournalSet 83.3%
Endjin.FreeAgent.Domain.JournalSetRoot 50%
Endjin.FreeAgent.Domain.Link 0%
Endjin.FreeAgent.Domain.Mileage 0%
Endjin.FreeAgent.Domain.MileageRoot 0%
Endjin.FreeAgent.Domain.MileagesRoot 0%
Endjin.FreeAgent.Domain.NoteItem 0%
Endjin.FreeAgent.Domain.NotesRoot 0%
Endjin.FreeAgent.Domain.OpeningBalance 0%
Endjin.FreeAgent.Domain.OpeningBalanceEntry 0%
Endjin.FreeAgent.Domain.OpeningBalanceJournal 0%
Endjin.FreeAgent.Domain.OpeningBalanceRoot 0%
Endjin.FreeAgent.Domain.PaymentMethods 0%
Endjin.FreeAgent.Domain.PayrollPayment 0%
Endjin.FreeAgent.Domain.PayrollPaymentRoot 0%
Endjin.FreeAgent.Domain.PayrollPaymentsRoot 0%
Endjin.FreeAgent.Domain.PayrollProfile 0%
Endjin.FreeAgent.Domain.PayrollProfileRoot 0%
Endjin.FreeAgent.Domain.PayrollProfilesRoot 0%
Endjin.FreeAgent.Domain.Payslip 0%
Endjin.FreeAgent.Domain.PayslipRoot 0%
Endjin.FreeAgent.Domain.PayslipsRoot 0%
Endjin.FreeAgent.Domain.ProfitAndLoss 0%
Endjin.FreeAgent.Domain.ProfitAndLossEntry 0%
Endjin.FreeAgent.Domain.ProfitAndLossRoot 0%
Endjin.FreeAgent.Domain.Project 100%
Endjin.FreeAgent.Domain.ProjectRoot 50%
Endjin.FreeAgent.Domain.ProjectsRoot 50%
Endjin.FreeAgent.Domain.PurchaseAgedCreditors 0%
Endjin.FreeAgent.Domain.PurchaseAgedCreditorsRoot 0%
Endjin.FreeAgent.Domain.RecurringInvoice 0%
Endjin.FreeAgent.Domain.RecurringInvoiceRoot 0%
Endjin.FreeAgent.Domain.RecurringInvoicesRoot 0%
Endjin.FreeAgent.Domain.SalesAgedDebtors 0%
Endjin.FreeAgent.Domain.SalesAgedDebtorsRoot 0%
Endjin.FreeAgent.Domain.SalesTaxRate 0%
Endjin.FreeAgent.Domain.SalesTaxRatesRoot 0%
Endjin.FreeAgent.Domain.SelfAssessmentReturn 0%
Endjin.FreeAgent.Domain.SelfAssessmentReturnFiling 0%
Endjin.FreeAgent.Domain.SelfAssessmentReturnFilingRoot 0%
Endjin.FreeAgent.Domain.SelfAssessmentReturnRoot 0%
Endjin.FreeAgent.Domain.SelfAssessmentReturnsRoot 0%
Endjin.FreeAgent.Domain.StatementUpload 0%
Endjin.FreeAgent.Domain.StatementUploadRoot 0%
Endjin.FreeAgent.Domain.StockItem 0%
Endjin.FreeAgent.Domain.StockItemRoot 0%
Endjin.FreeAgent.Domain.StockItemsRoot 0%
Endjin.FreeAgent.Domain.TaskItem 90%
Endjin.FreeAgent.Domain.TaskRoot 50%
Endjin.FreeAgent.Domain.TasksRoot 50%
Endjin.FreeAgent.Domain.Timeslip 100%
Endjin.FreeAgent.Domain.TimeslipsRoot 50%
Endjin.FreeAgent.Domain.TrialBalance 0%
Endjin.FreeAgent.Domain.TrialBalanceEntry 0%
Endjin.FreeAgent.Domain.TrialBalanceRoot 0%
Endjin.FreeAgent.Domain.User 100%
Endjin.FreeAgent.Domain.UserRoot 50%
Endjin.FreeAgent.Domain.UsersRoot 50%
Endjin.FreeAgent.Domain.VatReturn 94.4%
Endjin.FreeAgent.Domain.VatReturnFiling 75%
Endjin.FreeAgent.Domain.VatReturnFilingRoot 50%
Endjin.FreeAgent.Domain.VatReturnRoot 50%
Endjin.FreeAgent.Domain.VatReturnsRoot 50%
Endjin.FreeAgent.Domain.Webhook 0%
Endjin.FreeAgent.Domain.WebhookPayload 0%
Endjin.FreeAgent.Domain.WebhookRoot 0%
Endjin.FreeAgent.Domain.WebhooksRoot 0%

@JamesDawson JamesDawson added the no_release Suppresses auto_release functionality label Nov 13, 2025
@github-actions

Copy link
Copy Markdown

Test Results

194 tests  ±0   194 ✅ ±0   4s ⏱️ -1s
  2 suites ±0     0 💤 ±0 
  2 files   ±0     0 ❌ ±0 

Results for commit 0fcee1a. ± Comparison against base commit fecfc3e.

Copy link
Copy Markdown
Contributor

@Copilot mark the PR as ready to review

Copilot AI commented Nov 13, 2025

Copy link
Copy Markdown
Author

@Copilot mark the PR as ready to review

I don't have the ability to mark PRs as ready for review through GitHub's API. You'll need to change the PR status manually through the GitHub UI or using the gh CLI tool.

@JamesDawson
JamesDawson marked this pull request as ready for review November 13, 2025 13:57
Copilot AI requested a review from JamesDawson November 13, 2025 13:58
@JamesDawson
JamesDawson enabled auto-merge (squash) November 13, 2025 14:00
@JamesDawson
JamesDawson merged commit e55acfe into main Nov 13, 2025
9 checks passed
@JamesDawson
JamesDawson deleted the copilot/update-build-workflow-trigger branch November 13, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no_release Suppresses auto_release functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants