Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: winglang/wing
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.85.34
Choose a base ref
...
head repository: winglang/wing
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
  • 8 commits
  • 26 files changed
  • 3 contributors

Commits on Jan 2, 2025

  1. fix(sdk): logformat used in awscdk lambda functions is deprecated (#7228

    )
    
    The LogFormat used in AWS CDK for Lambda Functions is deprecated, and is being changed to LoggingFormat.
    
    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [ ] Tests added (always)
    - [ ] Docs updated (only required for features)
    - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    marciocadev authored Jan 2, 2025
    Configuration menu
    Copy the full SHA
    7aa9d6a View commit details
    Browse the repository at this point in the history

Commits on Jan 5, 2025

  1. fix(sdk): test execution in aws-cdk (#7232)

    The Pull Request v0.79.13 changed the way the lambdas test ARNs are passed to the TestRunnerClient, unfortunately, it was not noticed that the target awscdk should also be changed. This change allowed the tests to be executed correctly.
    
    Closes #7229
    
    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [ ] Tests added (always)
    - [ ] Docs updated (only required for features)
    - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    marciocadev authored Jan 5, 2025
    Configuration menu
    Copy the full SHA
    4315ce3 View commit details
    Browse the repository at this point in the history
  2. feat(sdk): adding cloud.Service to awscdk (#7230)

    # 👷 WORK IN PROGRESS ... AGAIN 🏗️
    
    Adding `cloud.Service` to AWS-CDK
    
    Closes #7226 
    
    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [x] Tests added (always)
    - [ ] Docs updated (only required for features)
    - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    marciocadev authored Jan 5, 2025
    Configuration menu
    Copy the full SHA
    332bd69 View commit details
    Browse the repository at this point in the history

Commits on Jan 6, 2025

  1. fix(sdk): some fixes and minor refactoring (#7234)

    I made two small corrections and a refactoring in this PR:
    
    1. There was an error in an environment variable related to the bucket name that was passed in the aws-cdk build.
    2. When updating the lambda function log, I assigned the new enum to the old variable, but I have now corrected it to the proper variable.
    3. It didn't make sense to have a file just to keep a global variable related to the Counter, so I removed the file and moved the global variable to the shared-aws/counter.ts file.
    
    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [ ] Tests added (always)
    - [ ] Docs updated (only required for features)
    - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    marciocadev authored Jan 6, 2025
    Configuration menu
    Copy the full SHA
    680203a View commit details
    Browse the repository at this point in the history

Commits on Jan 7, 2025

  1. feat(docs): add toolchain validation to contrib docs (#7227)

    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [x] Tests added (always)
    - [x] Docs updated (only required for features)
    - [x] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    eladb authored Jan 7, 2025
    Configuration menu
    Copy the full SHA
    dc35958 View commit details
    Browse the repository at this point in the history
  2. chore: replace mergify's queue_rules.speculative_checks in favor of…

    … `merge_queue.max_parallel_checks` (#7238)
    skyrpex authored Jan 7, 2025
    Configuration menu
    Copy the full SHA
    bb7f1ba View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    9828ae6 View commit details
    Browse the repository at this point in the history
  4. fix(sdk): cloud.Service test when using tf-aws target (#7233)

    I try to run this test whit the command `wing test --platform tf-aws minimal.test.w`
    
    ```w
    bring cloud;
    
    let s = new cloud.Service(inflight () => {
      log("hello, service!");
    
      return () => {
        log("stopping!");
      };
    });
    
    test "start and stop" {
      assert(s.started());
      s.stop();
      assert(!s.started());
    }
    ```
    
    but I get this error
    
    ```shell
    $ wing test --platform tf-aws minimal.test.w
    ✔ Compiling minimal.test.w to tf-aws...
    
    ✔ terraform init
    
    ✖ terraform apply
    
    Command failed: terraform apply -auto-approve
    ╷
    │ Error: creating ECS Cluster (Test.mfet8zI2Z4_cluster): InvalidParameterException: Cluster name must match ^[a-zA-Z0-9\-_]{1,255}$, but was Test.mfet8zI2Z4_cluster
    │
    │   with aws_ecs_cluster.Testmfet8zI2Z4_env0_Service_ECSCluster_1D77BD98,
    │   on main.tf.json line 116, in resource.aws_ecs_cluster.Testmfet8zI2Z4_env0_Service_ECSCluster_1D77BD98:
    │  116:       }
    │
    ╵
    
    ✔ terraform destroy
    
    
    Tests 1 failed (1)
    Snapshots 1 skipped
    Test Files 1 failed (1)
    Duration 3m29.12s
    ```
    
    When objects are generated for the test, the default is to generate the cluster name like this Test.mfet8zI2Z4_cluster. 
    However, the cluster name cannot have a `.`, so I made a small adjustment to replace `.` with `_` to avoid this error
    
    
    ## Checklist
    
    - [x] Title matches [Winglang's style guide](https://www.winglang.io/contributing/start-here/pull_requests#how-are-pull-request-titles-formatted)
    - [x] Description explains motivation and solution
    - [ ] Tests added (always)
    - [ ] Docs updated (only required for features)
    - [ ] Added `pr/e2e-full` label if this feature requires end-to-end testing
    
    *By submitting this pull request, I confirm that my contribution is made under the terms of the [Wing Cloud Contribution License](https://github.com/winglang/wing/blob/main/CONTRIBUTION_LICENSE.md)*.
    marciocadev authored Jan 7, 2025
    Configuration menu
    Copy the full SHA
    c10997b View commit details
    Browse the repository at this point in the history
Loading