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.

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

Revert "DQ-384 feat(contract): add support for contract impact analysis" #137

Merged
merged 1 commit into from
Oct 7, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions .github/workflows/test-action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ name: Test Action
on:
pull_request:
types: [opened, edited, synchronize, reopened, closed]
paths:
- 'test/**'

jobs:
get-downstream-assets:
Expand All @@ -31,4 +29,3 @@ jobs:
beta: Wide World Importers PE1
test-action: Wide World Importers PE1
IGNORE_MODEL_ALIAS_MATCHING: true
ATLAN_CONFIG: .atlan/config.yaml
5 changes: 1 addition & 4 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,4 @@
node_modules/
event.json
.idea
.DS_Store
.vscode/
contracts/
.atlan/
.DS_Store
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ After you've completed the configuration above, create a pull request with a cha
| `ATLAN_API_TOKEN` | Needed for authenticating API requests to the user's tenant. https://ask.atlan.com/hc/en-us/articles/8312649180049 | true |
| `DBT_ENVIRONMENT_BRANCH_MAP` | Map Github branch with specific dbt environment, if you do this - Atlan Github action will pick lineage for that specific environment from Atlan.You can provide the mapping like `branch name`: `dbt environment name`. <br><br>main: DBT-DEMO-PROD<br>beta: Wide World Importers PE1<br>test-action: Wide World Importers PE1 | false |
| `IGNORE_MODEL_ALIAS_MATCHING` | By default the action checks if there's an alias defined for a model in the code and looks for the relevant asset in Atlan using that alias. You can turn off matching alias name using this variable. | false | false |
| `ATLAN_CONFIG` | The Atlan CLI configuration file is typically located at `.atlan/config.yaml`. Setting the `ATLAN_CONFIG` environment variable will trigger impact analysis on Atlan Data Contracts, if included in a GitHub pull request. | false | |

## FAQs

Expand Down
3 changes: 0 additions & 3 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ inputs:
description: "Ignore model alias matching"
required: false
default: false
ATLAN_CONFIG:
description: "Atlan CLI config file location"
required: false
runs:
using: "node16"
main: "dist/index.js"
Expand Down
33 changes: 0 additions & 33 deletions adapters/api/get-asset-classifications.js

This file was deleted.

7 changes: 3 additions & 4 deletions adapters/api/get-classifications.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
import fetch from "node-fetch";
import {
ATLAN_API_TOKEN,
ATLAN_INSTANCE_URL,
ATLAN_API_TOKEN,
} from "../utils/get-environment-variables.js";

import fetch from "node-fetch";

export default async function getClassifications({
sendSegmentEventOfIntegration,
}) {
Expand Down Expand Up @@ -35,4 +34,4 @@ export default async function getClassifications({
});

return response?.classificationDefs;
}
}
102 changes: 0 additions & 102 deletions adapters/api/get-contract-asset.js

This file was deleted.

Loading
Loading