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

Breaking change: refactor for grpc client #37

Merged
merged 31 commits into from
Feb 4, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
c941638
Use DI for deserialization
withinboredom Jan 29, 2021
fafa385
Refactor serialization
withinboredom Jan 30, 2021
c2e982c
Get state working
withinboredom Jan 30, 2021
26bd1e3
Add transactional state
withinboredom Jan 30, 2021
8059b31
Actor state updated
withinboredom Jan 30, 2021
aff004a
Get actors working
withinboredom Jan 30, 2021
cd3ff78
Reformat all
withinboredom Jan 30, 2021
37a8f1a
Get almost everything working
withinboredom Jan 30, 2021
18aa387
Update runtime
withinboredom Jan 30, 2021
397edb7
Fix generators
withinboredom Jan 30, 2021
dd1427a
Clean things up
withinboredom Jan 30, 2021
55a1c3f
Add actor endpoints
withinboredom Jan 31, 2021
7be27c9
Get actor integration tests working
withinboredom Jan 31, 2021
9d29d83
Test cloud events and state
withinboredom Jan 31, 2021
9ee9f5a
Get all integration tests to pass
withinboredom Feb 1, 2021
dad8407
Remove now-defunct runtime
withinboredom Feb 1, 2021
df2f563
Cleaning up
withinboredom Feb 1, 2021
cddff4f
Update unit tests
withinboredom Feb 1, 2021
0ff3af9
add more tests
withinboredom Feb 1, 2021
3b8b980
Start wiring up configs
withinboredom Feb 2, 2021
88d56c3
Update readme and config
withinboredom Feb 2, 2021
ca1509f
Fix some minor issues
withinboredom Feb 2, 2021
19010f5
Allow pure attribute in tests
withinboredom Feb 2, 2021
3963154
port is a string
withinboredom Feb 2, 2021
7cf3552
Update the docs
withinboredom Feb 2, 2021
daa77ad
Add run function
withinboredom Feb 2, 2021
30c388c
make sure default value is last in the chain
withinboredom Feb 2, 2021
e1b080d
Make parameters optional
withinboredom Feb 2, 2021
27bf4b3
ensure a request exists
withinboredom Feb 2, 2021
5f15637
Fix deserializer instantiating a weird obj
withinboredom Feb 2, 2021
8f32790
optimizations
withinboredom Feb 3, 2021
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
11 changes: 4 additions & 7 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -1,26 +1,23 @@
---
name: Bug report
about: Report a bug in Dapr
title: ''
labels: kind/bug
assignees: ''
name: Bug report about: Report a bug in Dapr title: ''
labels: kind/bug assignees: ''

---

## Expected Behavior

<!-- Briefly describe what you expect to happen -->


## Actual Behavior

<!-- Briefly describe what is actually happening -->


## Steps to Reproduce the Problem

<!-- How can a maintainer reproduce this issue (be detailed) -->

## Release Note

<!-- How should the fix for this issue be communicated in our release notes? It can be populated later. -->
<!-- Keep it as a single line. Examples: -->

Expand Down
7 changes: 2 additions & 5 deletions .github/ISSUE_TEMPLATE/discussion.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
---
name: Discussion
about: Start a discussion for Dapr
title: ''
labels: kind/discussion
assignees: ''
name: Discussion about: Start a discussion for Dapr title: ''
labels: kind/discussion assignees: ''

---
9 changes: 4 additions & 5 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,13 @@
---
name: Feature Request
about: Create a Feature Request for Dapr
title: ''
labels: kind/enhancement
assignees: ''
name: Feature Request about: Create a Feature Request for Dapr title: ''
labels: kind/enhancement assignees: ''

---

## Describe the feature

## Release Note

<!-- How should this new feature be announced in our release notes? It can be populated later. -->
<!-- Keep it as a single line. Examples: -->

Expand Down
8 changes: 3 additions & 5 deletions .github/ISSUE_TEMPLATE/proposal.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
---
name: Proposal
about: Create a proposal for Dapr
title: ''
labels: kind/proposal
assignees: ''
name: Proposal about: Create a proposal for Dapr title: ''
labels: kind/proposal assignees: ''

---

## Describe the proposal
8 changes: 3 additions & 5 deletions .github/ISSUE_TEMPLATE/question.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
---
name: Question
about: Ask a question about Dapr
title: ''
labels: kind/question
assignees: ''
name: Question about: Ask a question about Dapr title: ''
labels: kind/question assignees: ''

---

## Ask your question here
3 changes: 2 additions & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ _Please explain the changes you've made_

## Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to
implementation.

Please reference the issue this PR will close: #_[issue number]_

Expand Down
58 changes: 29 additions & 29 deletions .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,35 +11,35 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2
- uses: shivammathur/[email protected]
with:
php-version: 8.0
extensions: xdebug
- name: Validate composer.json and composer.lock
run: composer validate

- name: Cache Composer packages
id: composer-cache
uses: actions/cache@v2
with:
path: vendor
key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
restore-keys: |
${{ runner.os }}-php-

- name: Install dependencies
if: steps.composer-cache.outputs.cache-hit != 'true'
run: composer install --prefer-dist --no-progress --no-suggest

# Add a test script to composer.json, for instance: "test": "vendor/bin/phpunit"
# Docs: https://getcomposer.org/doc/articles/scripts.md

- name: Run test suite
run: composer run-script test

- name: upload coverage
run: bash <(curl -s https://codecov.io/bash)
- uses: actions/checkout@v2
- uses: shivammathur/[email protected]
with:
php-version: 8.0
extensions: xdebug
- name: Validate composer.json and composer.lock
run: composer validate

- name: Cache Composer packages
id: composer-cache
uses: actions/cache@v2
with:
path: vendor
key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
restore-keys: |
${{ runner.os }}-php-

- name: Install dependencies
if: steps.composer-cache.outputs.cache-hit != 'true'
run: composer install --prefer-dist --no-progress --no-suggest

# Add a test script to composer.json, for instance: "test": "vendor/bin/phpunit"
# Docs: https://getcomposer.org/doc/articles/scripts.md

- name: Run test suite
run: composer run-script test

- name: upload coverage
run: bash <(curl -s https://codecov.io/bash)
lint:
runs-on: ubuntu-latest

Expand Down
8 changes: 4 additions & 4 deletions components/pubsub.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
type: pubsub.redis
metadata:
- name: redisHost
value: localhost:6379
- name: redisPassword
value: ""
- name: redisHost
value: localhost:6379
- name: redisPassword
value: ""
12 changes: 6 additions & 6 deletions components/statestore.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@ metadata:
spec:
type: state.redis
metadata:
- name: redisHost
value: localhost:6379
- name: redisPassword
value: ""
- name: actorStateStore
value: "true"
- name: redisHost
value: localhost:6379
- name: redisPassword
value: ""
- name: actorStateStore
value: "true"
14 changes: 10 additions & 4 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,19 @@
"ext-mbstring": "*",
"ext-curl": "*",
"psr/log": "^1.1",
"nette/php-generator": "^3.5"
"nette/php-generator": "^3.5",
"php-di/php-di": "^6.3",
"monolog/monolog": "^2.2",
"nikic/fast-route": "^1.3",
"nyholm/psr7": "^1.3",
"nyholm/psr7-server": "^1.0",
"php-di/invoker": "^2.3",
"laminas/laminas-httphandlerrunner": "^1.3"
},
"require-dev": {
"ext-xdebug": "*",
"vimeo/psalm": "^4.3",
"phpunit/phpunit": "^9",
"monolog/monolog": "^2.2"
"phpunit/phpunit": "^9"
},
"autoload": {
"psr-4": {
Expand All @@ -33,7 +39,7 @@
"test": "./vendor/bin/phpunit --testdox tests --coverage-clover coverage.xml tests --whitelist src",
"start": [
"PHP_CLI_SERVER_WORKERS=100 dapr run --components-path components -a dev -p 9502 -- php -S 0.0.0.0:9502 -t src"
],
],
"dapr-init": [
"dapr init --runtime-version 1.0.0-rc.3"
],
Expand Down
Loading