Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: Adds GitHub Action to create automated Release PRs #865

Merged
merged 7 commits into from May 13, 2024
Merged

ci: Adds GitHub Action to create automated Release PRs #865

merged 7 commits into from May 13, 2024

Conversation

frtibble
Copy link
Contributor

@frtibble frtibble commented May 9, 2024 •
edited

Purpose

This PR adds a GitHub Action which will automatically create Release PRs once new PRs are merged into main. These can then be published.

Note: Before this PR is merged I will enable "Releases" in the GitHub repository, which is a small settings change.
I have created the access token required by the Action, and have documented those steps in Release Guidelines.

Resolves #837
Resolves #838

Does this introduce a breaking change?

  • Yes
  • No

How to Test

I have tested this in a personal repo, once this PR is merged it should create a Release PR. If it does not, this will not block anything else in the repo.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 9, 2024 •
edited

Coverage

Coverage Report
FileStmtsMissCoverMissing
code
   app.py14140%1–4, 6–7, 9, 12–14, 16, 18, 20–21
   create_app.py130397%190, 195, 313
code/backend
   Admin.py21210%1–5, 7, 9–10, 12, 15–16, 18–19, 22, 29, 36, 39–41, 43, 45
code/backend/batch
   AddURLEmbeddings.py470100% 
   BatchPushResults.py250100% 
   BatchStartProcessing.py180100% 
   GetConversationResponse.py320100% 
   function_app.py16160%1–8, 10, 12–13, 15, 18–21
code/backend/batch/utilities/common
   Answer.py24195%39
   SourceDocument.py60591%44, 47, 51, 55, 128
code/backend/batch/utilities/document_chunking
   __init__.py70100% 
   chunking_strategy.py15193%25
   document_chunking_base.py10280%10, 16
   fixed_size_overlap.py190100% 
   layout.py190100% 
   page.py170100% 
   paragraph.py9277%9, 15
   strategies.py15380%15–16, 18
code/backend/batch/utilities/document_loading
   DocumentLoadingBase.py9188%13
   Layout.py12558%9, 12–13, 16, 25
   Read.py12558%9, 12–13, 16, 25
   Strategies.py20575%17, 19, 22–23, 25
   Web.py19194%23
   WordDocument.py251348%11–12, 22–24, 27, 30, 33–37, 45
   __init__.py15193%16
code/backend/batch/utilities/helpers
   AzureBlobStorageClient.py803062%22–24, 32, 52, 55–56, 61, 65, 90–91, 93, 97, 153, 156, 175, 178, 180, 188–192, 215, 219–223, 225
   AzureFormRecognizerHelper.py817013%11, 13, 16–17, 25, 27, 44–45, 52–55, 60–68, 73–75, 77–78, 81, 84–86, 88–90, 93, 97–98, 105–109, 111–114, 117–131, 133, 135–137, 139–140, 143, 145–147
   AzureSearchHelper.py470100% 
   DocumentChunkingHelper.py13192%19
   DocumentLoadingHelper.py12191%14
   EnvHelper.py1241091%201, 206–207, 210–212, 221, 225–227
   LLMHelper.py421173%40–41, 50, 61–62, 73, 86–87, 94, 113, 121
   OrchestratorHelper.py13469%22–24, 27
code/backend/batch/utilities/helpers/config
   ConfigHelper.py1230100% 
   EmbeddingConfig.py12191%27
code/backend/batch/utilities/helpers/embedders
   EmbedderBase.py5180%7
   EmbedderFactory.py10370%12–13, 15
   IntegratedVectorizationEmbedder.py31487%20, 39–41
   PushEmbedder.py460100% 
code/backend/batch/utilities/integrated_vectorization
   AzureSearchDatasource.py190100% 
   AzureSearchIndex.py350100% 
   AzureSearchIndexer.py20290%47–48
   AzureSearchSkillset.py220100% 
code/backend/batch/utilities/loggers
   ConversationLogger.py362822%8, 11–12, 15–24, 27–30, 33–42, 46
code/backend/batch/utilities/orchestrator
   LangChainAgent.py601968%21–24, 26, 61–62, 82–85, 102–103, 106–109, 116–117
   OpenAIFunctions.py543633%62, 64, 70–73, 75, 78, 85–89, 93–94, 96, 102–106, 110–112, 115, 118–119, 122, 127–129, 132–134, 137, 142
   OrchestrationStrategy.py50100% 
   OrchestratorBase.py50884%33, 72–74, 81–83, 93
   SemanticKernel.py540100% 
   Strategies.py12741%8–13, 15
   __init__.py110100% 
code/backend/batch/utilities/parser
   OutputParserTool.py390100% 
   ParserBase.py9277%9, 19
   __init__.py7271%7, 11
code/backend/batch/utilities/plugins
   ChatPlugin.py150100% 
   PostAnsweringPlugin.py80100% 
code/backend/batch/utilities/search
   AzureSearchHandler.py58296%24, 30
   IntegratedVectorizationSearchHandler.py61395%35, 111, 120
   Search.py13192%15
   SearchHandlerBase.py361363%13–15, 18–20, 24, 28, 32, 36, 40, 44, 48
code/backend/batch/utilities/tools
   AnswerProcessingBase.py8275%8, 12
   AnsweringToolBase.py9277%9, 15
   ContentSafetyChecker.py412539%16, 18–19, 24, 30–32, 35–36, 42–43, 49–54, 57–59, 61, 65–67, 69
   PostPromptTool.py221340%11, 14–15, 17–18, 22, 29, 36–37, 45, 51–52, 60
   QuestionAnswerTool.py610100% 
   TextProcessingTool.py16943%9, 12–15, 21, 23, 30, 37
code/backend/pages
   01_Ingest_Data.py70700%1–10, 12–14, 16, 22, 29, 32–33, 36–39, 41–44, 48–50, 53–55, 58–65, 68–70, 72, 75–78, 81, 86–88, 90–91, 93–94, 97–98, 102–104, 109–112, 119–120, 125, 131–132
   02_Explore_Data.py26260%1–7, 9–10, 12, 18, 25, 28, 36, 38–39, 41–44, 46–49, 52–53
   03_Delete_Data.py36360%1–7, 9–11, 13, 19, 26, 29, 37, 39–40, 42–45, 47, 49–50, 54, 58–62, 64, 67–68, 70–72
   04_Configuration.py1351350%1–9, 11–12, 14, 21, 28, 30, 35–44, 47–48, 51–62, 64–65, 67–69, 73–74, 86–90, 93–94, 98–100, 103–104, 107–108, 111–112, 135, 137–138, 140–144, 146–149, 152–156, 163–164, 174–176, 178, 198–199, 201, 203, 209, 217, 225, 232–233, 240, 242–243, 247, 255, 261, 268, 286–290, 296–297, 316–317, 321, 323–324, 347, 381–382, 384–385, 388–389, 392–395, 397–398, 400–402, 404–407, 409–410
TOTAL229767670% 

Tests Skipped Failures Errors Time
178 0 💤 0 ❌ 0 🔥 10.573s ⏱️

cecheta
cecheta reviewed May 9, 2024
View reviewed changes
.github/workflows/release-please.yml Show resolved Hide resolved
docs/RELEASE_GUIDELINES.md Outdated Show resolved Hide resolved
cecheta
cecheta approved these changes May 10, 2024
View reviewed changes
Copy link
Collaborator

@cecheta cecheta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 😀

gaurarpit
gaurarpit approved these changes May 10, 2024
View reviewed changes
Copy link
Collaborator

@gaurarpit gaurarpit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving, but, have 1 question - left in comment thread.

@frtibble frtibble added this pull request to the merge queue May 13, 2024
Merged via the queue into Azure-Samples:main with commit 27e1e06 May 13, 2024
6 checks passed
@frtibble frtibble deleted the release-please branch May 13, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@superhindupur superhindupur superhindupur left review comments

@cecheta cecheta cecheta approved these changes

@gaurarpit gaurarpit gaurarpit approved these changes

@ross-p-smith ross-p-smith ross-p-smith approved these changes

@liammoat liammoat Awaiting requested review from liammoat

@adamdougal adamdougal Awaiting requested review from adamdougal

@komalg1 komalg1 Awaiting requested review from komalg1

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Release Guidelines to document release process Add GitHub action to create a release
5 participants
@frtibble @superhindupur @gaurarpit @ross-p-smith @cecheta