Skip to content

Commit

Permalink
Post-merge fixes (#324)
Browse files Browse the repository at this point in the history
* Fixes merge errors from #323 and removes components deleted in Steeltoe v4

* Add target branch

* include v4 in metadata

---------

Co-authored-by: Tim Hess <tim.hess@broadcom.com>
  • Loading branch information
bart-vmware and TimHess authored May 28, 2024
1 parent 9b7f19c commit d6abd05
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 97 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/build-and-stage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ on:
pull_request:
branches:
- main
- v4
push:
branches:
- main
Expand All @@ -21,6 +22,7 @@ env:
DOCFX_SOURCE_BRANCH_NAME: ${{ github.base_ref || github.ref_name }}
STEELTOE_V2_VERSION: 2.5.5
STEELTOE_V3_VERSION: 3.2.6
STEELTOE_V4_VERSION: main
SITE_IMAGE_VERSION: ${{ github.event_name == 'pull_request' && format('pr-{0}', github.event.number) || github.run_id }}

jobs:
Expand Down Expand Up @@ -66,7 +68,7 @@ jobs:
echo "metadata-image-version=${{ env.STEELTOE_VERSIONS }}" >> "$GITHUB_OUTPUT"
fi
env:
STEELTOE_VERSIONS: ${{ env.STEELTOE_V2_VERSION }}-${{ env.STEELTOE_V3_VERSION }}
STEELTOE_VERSIONS: ${{ env.STEELTOE_V2_VERSION }}-${{ env.STEELTOE_V3_VERSION }}-${{ env.STEELTOE_V4_VERSION }}
shell: bash

build-push-docfx:
Expand Down Expand Up @@ -108,6 +110,7 @@ jobs:
run: |-
echo '2:'${{ env.STEELTOE_V2_VERSION }} > metadata.conf
echo '3:'${{ env.STEELTOE_V3_VERSION }} >> metadata.conf
echo '4:'${{ env.STEELTOE_V4_VERSION }} >> metadata.conf
shell: bash

- name: Login to container registry
Expand Down
15 changes: 4 additions & 11 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,16 +13,7 @@ This is the home of Steeltoe documentation and blog articles. The site uses [Doc
| `/guides` | guides for getting started with Steeltoe
| `/template` | theming

- /api: holds the project documentation markdown and table of contents
- /v2: version 2 documentation
- /all: this holds generated API docs from doc-comments in source code
- /v3: version 3 documentation
- /all: this holds generated API docs from doc-comments in source code
- /v4: version 4 documentation
- /all: this holds generated API docs from doc-comments in source code
- /articles: holds the markdown for blog posts
- /images: the images
- /template/steeltoe: odd files that overwrite the default DocFX theme
## DocFX Markdown

DocFX offers an enhanced flavor of Markdown. To see examples and learn more, view the [DocFX Flavored Markdown](https://dotnet.github.io/docfx/spec/docfx_flavored_markdown.html) documentation.

Expand Down Expand Up @@ -131,7 +122,7 @@ For working on any non-trivial changes, there are several ways to build and run

The easiest way to build and run the site is this command: `docfx build --serve --port 8082`.

### Build API docs for Steeltoe 2 and 3
### Build API docs for Steeltoe 2, 3 and 4

Building the API docs is not required for the site to run locally.

Expand All @@ -140,9 +131,11 @@ If needed, these commands will download the Steeltoe source code and generate AP
```bash
git clone https://github.com/SteeltoeOSS/Steeltoe sources/v2 -b release/2.5
git clone https://github.com/SteeltoeOSS/Steeltoe sources/v3 -b release/3.2
git clone https://github.com/SteeltoeOSS/Steeltoe sources/v4 -b release/main
git clean -fX api
docfx metadata api-v2.json
docfx metadata api-v3.json
docfx metadata api-v4.json
docfx metadata api-all.json
```

Expand Down
57 changes: 0 additions & 57 deletions api-v4.json
Original file line number Diff line number Diff line change
Expand Up @@ -95,25 +95,6 @@
},
"version": "4.x"
},
{
"src": [
{
"files": [
"**/*.csproj"
],
"exclude": [ "**/bin/**", "**/obj/**" ],
"src": "sources/v4/src/Integration/src"
}
],
"dest": "api/browser/v4/integration",
"disableGitFeatures": false,
"disableDefaultFilter": false,
"filter": "api-filter.yml",
"properties": {
"TargetFramework": "net6.0"
},
"version": "4.x"
},
{
"src": [
{
Expand Down Expand Up @@ -152,25 +133,6 @@
},
"version": "4.x"
},
{
"src": [
{
"files": [
"**/*.csproj"
],
"exclude": [ "**/bin/**", "**/obj/**" ],
"src": "sources/v4/src/Messaging/src"
}
],
"dest": "api/browser/v4/messaging",
"disableGitFeatures": false,
"disableDefaultFilter": false,
"filter": "api-filter.yml",
"properties": {
"TargetFramework": "net6.0"
},
"version": "4.x"
},
{
"src": [
{
Expand All @@ -189,25 +151,6 @@
"TargetFramework": "net6.0"
},
"version": "4.x"
},
{
"src": [
{
"files": [
"**/*.csproj"
],
"exclude": [ "**/bin/**", "**/obj/**" ],
"src": "sources/v4/src/Stream/src"
}
],
"dest": "api/browser/v4/stream",
"disableGitFeatures": false,
"disableDefaultFilter": false,
"filter": "api-filter.yml",
"properties": {
"TargetFramework": "net6.0"
},
"version": "4.x"
}
]
}
20 changes: 1 addition & 19 deletions api/browser/v4/all/index.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,7 @@ Contains connectors for simplifying the process of connecting to backing service

Contains packages for interacting with service registries.

[view](/api/browser/v4/discovery/Steeltoe.Discovery.html)

## Steeltoe Integration

Contains packages for messaging integrations.

[view](/api/browser/v4/integration/Steeltoe.Integration.html)
[view](/api/browser/v4/discovery/Steeltoe.Discovery.HttpClients.html)

## Steeltoe Logging Extensions

Expand All @@ -59,20 +53,8 @@ Management tools to help manage and monitor your applications.

[view](/api/browser/v4/management/Steeltoe.Management.html)

## Steeltoe Messaging

Contains messaging support for RabbitMQ.

[view](/api/browser/v4/messaging/Steeltoe.Messaging.html)

## Steeltoe Security

Authentication and DataProtection libraries using security services on CloudFoundry.

[view](/api/browser/v4/security/Steeltoe.Security.Authentication.CloudFoundry.html)

## Steeltoe Stream

Contains streaming support for RabbitMQ.

[view](/api/browser/v4/stream/Steeltoe.Stream.Extensions.html)
3 changes: 2 additions & 1 deletion build-metadata.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,11 @@ get_sources sources/v4 $v4_sources
echo "building v2 metadata"
docfx metadata api-v2.json

get_sources sources/v3 $v3_sources
echo "building v3 metadata"
docfx metadata api-v3.json

echo "building v4 metadata"
docfx metadata api-v4.json

echo "building all metadata"
docfx metadata api-all.json
2 changes: 1 addition & 1 deletion main-site.localhost.json
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
{
"_rootHost": "http://localhost:8080"
"_rootHost": "https://localhost:8080"
}
6 changes: 3 additions & 3 deletions template/steeltoe/partials/navbar.tmpl.partial
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,12 @@
</button>
<div class="dropdown-menu" aria-labelledby="projects">
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/app-configuration" onClick="$(this).removeAttr('target')">Steeltoe Application Configuration</a>
<a class="dropdown-item" href="/api/v4/logging/" onClick="$(this).removeAttr('target')">Steeltoe Dynamic Logging</a>
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/cloud-management" onClick="$(this).removeAttr('target')">Steeltoe Management</a>
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/messaging" onClick="$(this).removeAttr('target')">Steeltoe Messaging</a>
<a class="dropdown-item" href="~/api/v4/fileshares" onClick="$(this).removeAttr('target')">Steeltoe Network File Shares</a>
<a class="dropdown-item" href="/api/v4/fileshares" onClick="$(this).removeAttr('target')">Steeltoe Network File Sharing</a>
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/security-providers" onClick="$(this).removeAttr('target')">Steeltoe Security</a>
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/service-connectors" onClick="$(this).removeAttr('target')">Steeltoe Service Connectors</a>
<a class="dropdown-item" href="{{#_rootHost}}{{_rootHost}}{{/_rootHost}}{{^_rootHost}}https://steeltoe.io{{/_rootHost}}/service-discovery" onClick="$(this).removeAttr('target')">Steeltoe Service Discovery</a>
<a class="dropdown-item" href="~/api/v4/stream" onClick="$(this).removeAttr('target')">Steeltoe Stream</a>
</div>
</li>
<li class="nav-item mx-xl-2">
Expand Down
5 changes: 1 addition & 4 deletions template/steeltoe/partials/toc.tmpl.partial
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,10 @@
<option value="/api/browser/v4/common/Steeltoe.Common.html">Steeltoe.Common</option>
<option value="/api/browser/v4/configuration/Steeltoe.Configuration.html">Steeltoe.Configuration</option>
<option value="/api/browser/v4/connectors/Steeltoe.Connectors.html">Steeltoe.Connectors</option>
<option value="/api/browser/v4/discovery/Steeltoe.Discovery.html">Steeltoe.Discovery</option>
<option value="/api/browser/v4/integration/Steeltoe.Integration.html">Steeltoe.Integration</option>
<option value="/api/browser/v4/discovery/Steeltoe.Discovery.HttpClients.html">Steeltoe.Discovery</option>
<option value="/api/browser/v4/logging/Steeltoe.Logging.html">Steeltoe.Logging</option>
<option value="/api/browser/v4/management/Steeltoe.Management.html">Steeltoe.Management</option>
<option value="/api/browser/v4/messaging/Steeltoe.Messaging.html">Steeltoe.Messaging</option>
<option value="/api/browser/v4/security/Steeltoe.Security.Authentication.CloudFoundry.html">Steeltoe.Security</option>
<option value="/api/browser/v4/stream/Steeltoe.Stream.Extensions.html">Steeltoe.Stream</option>
</select>
<select id="api-namespace-v3" class="form-control api-namespace hide">
<option value="/api/browser/v3/all/">All namespaces</option>
Expand Down

0 comments on commit d6abd05

Please sign in to comment.